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 769190 - Clutter stage is too big and events wrong under hidpi gnome on wayland
Clutter stage is too big and events wrong under hidpi gnome on wayland
Status: RESOLVED OBSOLETE
Product: clutter
Classification: Platform
Component: gdk
git master
Other Linux
: Normal normal
: ---
Assigned To: clutter-maint
clutter-maint
Depends on:
Blocks:
 
 
Reported: 2016-07-26 13:24 UTC by Gustavo Noronha (kov)
Modified: 2021-06-10 11:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot showing the problem (39.63 KB, image/png)
2016-07-26 13:24 UTC, Gustavo Noronha (kov)
  Details
gdk: Ensure surface buffer scale is set (1.46 KB, patch)
2016-11-07 21:57 UTC, Sjoerd Simons
none Details | Review
updated version of sjoerds patch (1.25 KB, patch)
2017-07-02 05:29 UTC, Andreas Henriksson
none Details | Review

Description Gustavo Noronha (kov) 2016-07-26 13:24:50 UTC
Created attachment 332141 [details]
Screenshot showing the problem

Running Gnome on Wayland with a 3200x1800 screen, clutter stages are being scaled twice, it seems, while events seem to be scaled only once.

tests/interactive/test-events is a good example of this. The screenshot shows how the window looks like when started under that environment. I marked the point where I have to click to highlight the red actor with a small pink circle.

Starting the test with CLUTTER_BACKEND=wayland makes it be sized properly and events are also landing where they are supposed to. Same thing for the x11 backend.

I tried this with clutter from master, this is under Fedora 24, GTK+ 3.20.6.
Comment 1 Sjoerd Simons 2016-11-07 21:57:11 UTC
Created attachment 339280 [details] [review]
gdk: Ensure surface buffer scale is set

GDK will only set the buffer scale for the surfaces it creates when
attaching an image or when using an egl window window. Neither will
happen for native clutter applications, thus directly set the buffer
scale from clutter in that case (so it's in sync with the rendering
scale)

Signed-off-by: Sjoerd Simons <sjoerd@luon.net>
Comment 2 Emmanuele Bassi (:ebassi) 2016-11-28 15:42:01 UTC
Review of attachment 339280 [details] [review]:

Looks okay.

::: clutter/gdk/clutter-stage-gdk.c
@@ +232,3 @@
+
+      surface = gdk_wayland_window_get_wl_surface (stage_gdk->window);
+      wl_surface_set_buffer_scale (surface,

It's unfortunate that gdk_wayland_window_get_wl_surface() does not return the surface with the appropriate buffer scale already — but I guess it's too late to change it, now.
Comment 3 Andreas Henriksson 2017-07-02 05:29:52 UTC
Created attachment 354784 [details] [review]
updated version of sjoerds patch

Had to update sjoerds patch to make it apply on 1.26.2 again without fuzz. Since review was ok, someone might want to push this?
Comment 4 André Klapper 2021-06-10 11:34:06 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version of clutter, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a ticket at
  https://gitlab.gnome.org/GNOME/clutter/-/issues/

Thank you for your understanding and your help.