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 620440 - Combo-boxes loose focus
Combo-boxes loose focus
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: .General
2.90.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2010-06-03 08:34 UTC by Marek Kašík
Modified: 2010-06-15 17:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Make GTK+ device grabs take precedence over GTK+ grabs (2.17 KB, patch)
2010-06-15 17:20 UTC, Carlos Garnacho
accepted-commit_now Details | Review

Description Marek Kašík 2010-06-03 08:34:23 UTC
After recent commit named "Merge the xi2-for-master branch" (SHA1 ID bd4609b14042a91646cd9057764eecfbc6faf42b) combo boxes stopped to work in the print dialog. They show up but I can not select anything.
Similar thing happens when I click on gtk3-demo -> Color Selector -> Change the above color -> eyedropper.
Comment 1 Carlos Garnacho 2010-06-15 17:20:18 UTC
Created attachment 163700 [details] [review]
Make GTK+ device grabs take precedence over GTK+ grabs

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=620440 (Reported by
Marek Kašík), where the print dialog modality would impair the pop-up
window from getting events. Device GTK+ grabs gain in specificness.
Comment 2 Matthias Clasen 2010-06-15 17:27:27 UTC
Review of attachment 163700 [details] [review]:

Ok, makes sense I guess. Sneaky way to introduce grab priorities :-)

::: gtk/gtkmain.c
@@ -1476,3 @@
-       *  then we send the event to the original event widget.
-       *  This is the key to implementing modality.
-       */

We probably want to keep this comment ?
Comment 3 Carlos Garnacho 2010-06-15 17:47:37 UTC
(In reply to comment #2)
> Review of attachment 163700 [details] [review]:
> 
> Ok, makes sense I guess. Sneaky way to introduce grab priorities :-)

It is :), all in all, modality is something collaborative UIs will have to handle carefully.

> 
> ::: gtk/gtkmain.c
> @@ -1476,3 @@
> -       *  then we send the event to the original event widget.
> -       *  This is the key to implementing modality.
> -       */
> 
> We probably want to keep this comment ?

Oh, indeed, just added this back and pushed.