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 727994 - Gtk Popover closes on click
Gtk Popover closes on click
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: GtkPopover
3.12.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2014-04-10 19:25 UTC by Corentin Noël
Modified: 2014-05-19 12:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
popover: Check for uncaught button events from children (1.34 KB, patch)
2014-05-19 12:00 UTC, Carlos Garnacho
committed Details | Review

Description Corentin Noël 2014-04-10 19:25:11 UTC
In every Popover in my desktop, clicking in a widget inside a popover does close it while performing an action (for example clicking a button or even clicking on insensitive widgets).

I'm using Ubuntu Trusty with the elementary-testing PPA (that comes with Gtk+-3.12.0)

If you need any other information, fell free to ask me.

This bug is also liked to this one on our bug tracker:
https://bugs.launchpad.net/maya/+bug/1300567 (a little video is showing the problem)
Comment 1 Matthias Clasen 2014-04-13 19:14:57 UTC
Not happening with the popovers I have access to for testing. A small usecase would be fantastic here. A few guesses: 
- are you stopping propagation of events after you've handled them ?
Comment 2 Victor Eduardo 2014-04-14 02:30:19 UTC
This usually happens when clicking on a GtkSwitch packed into a GtkPopover, or when clicking any insensitive child widget (so far I've only tested GtkScale and GtkComboBox).

FWIW the popovers are set to modal=true, and we use GtkMenuButton to show them.
Comment 3 Carlos Garnacho 2014-05-19 12:00:08 UTC
The following fix has been pushed:
cacccf7 popover: Check for uncaught button events from children
Comment 4 Carlos Garnacho 2014-05-19 12:00:12 UTC
Created attachment 276764 [details] [review]
popover: Check for uncaught button events from children

And deal correctly with those as a click within the popover.