After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 731268 - Events being clamped to the pre-fullscreen window size when stage goes fullscreen
Events being clamped to the pre-fullscreen window size when stage goes fullsc...
Status: RESOLVED FIXED
Product: clutter
Classification: Platform
Component: x11
1.18.x
Other Mac OS
: Normal normal
: ---
Assigned To: clutter-maint
clutter-maint
Depends on:
Blocks:
 
 
Reported: 2014-06-05 13:04 UTC by Gustavo Noronha (kov)
Modified: 2014-06-05 18:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
device-manager-xi2: use allocation for clamping (1.61 KB, patch)
2014-06-05 13:11 UTC, Gustavo Noronha (kov)
committed Details | Review

Description Gustavo Noronha (kov) 2014-06-05 13:04:33 UTC
When in-page video goes fullscreen on webkit clutter, a new stage is created to host the video actor, and made fullscreen. The controls are positioned at the bottom of the screen, and they get no events. It turns out that happens because the events coordinates are clamped to the original size (640x480) of the stage.
Comment 1 Gustavo Noronha (kov) 2014-06-05 13:11:38 UTC
Created attachment 277951 [details] [review]
device-manager-xi2: use allocation for clamping

The coordinates translated by the XI2 device manager were being clamped using
the X window size kept by StageX11. However, when the stage is fullscreen,
that size is not updated to the screen size, but kept the same in order to
allow going back to it when the stage goes out of fullscreen.
Comment 2 Emmanuele Bassi (:ebassi) 2014-06-05 14:10:19 UTC
Review of attachment 277951 [details] [review]:

looks good to me.

please, commit to the clutter-1.20 and clutter-1.18 branches.
Comment 3 Gustavo Noronha (kov) 2014-06-05 18:02:48 UTC
Comment on attachment 277951 [details] [review]
device-manager-xi2: use allocation for clamping

pushed as d0aaf89f45542006c02c9f5ae20e730e9a1ff4c3 to clutter-1.18

Attachment 277951 [details] pushed as d0aaf89 - device-manager-xi2: use allocation for clamping
Comment 4 Gustavo Noronha (kov) 2014-06-05 18:03:48 UTC
fcdd222 to clutter-1.20