<html> <head> <base href="https://bugzilla.gnome.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - [wayland] input shape and opaque region not applied without begin_paint()/end_paint()" href="https://bugzilla.gnome.org/show_bug.cgi?id=774534#c10">Comment # 10</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - [wayland] input shape and opaque region not applied without begin_paint()/end_paint()" href="https://bugzilla.gnome.org/show_bug.cgi?id=774534">bug 774534</a> from <a href="page.cgi?id=describeuser.html&login=ofourdan%40redhat.com" title="Olivier Fourdan <ofourdan@redhat.com>"> Olivier Fourdan</a> <pre> <jadahl> ofourdan: i see, could we make that explicit maybe? instead of sync... we do "wl_surface_set_input_region(empty)" with a comment saying why its empty and its fine? <ofourdan> jadahl: sorry I didn't quite get what you mean about making it explicit <jadahl> ofourdan: instead of using the sync function, just do it directly with the special case (i.e. empty region) so that its obvious that it won't break on resize <ofourdan> I still don't get what you mean, do you mean in gdk or in clutter? in clutter, we cannot as there is no api to retrieve the actual wayland subsurface from gdk <ofourdan> oh, you mean, same as now but only for the special case? <jadahl> ofourdan: i meant in gdk, yes. make the special case more obvious by making it explicit <jadahl> ofourdan: since the fact that it'll get set to no-input is hidden from the call site <jadahl> ofourdan: as a matter of fact, we could set the input region very early, so the first ever commit catches it (if there is any before set_desync)</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are on the CC list for the bug.</li> </ul> </body> </html>