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 598235 - Login window looses keyboard focus
Login window looses keyboard focus
Status: RESOLVED DUPLICATE of bug 592183
Product: gdm
Classification: Core
Component: general
2.28.x
Other Linux
: Normal normal
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2009-10-13 05:29 UTC by Robert Ancell
Modified: 2010-02-01 18:53 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Removed the DOCK hint from the login window (1.13 KB, patch)
2009-10-13 05:32 UTC, Robert Ancell
reviewed Details | Review

Description Robert Ancell 2009-10-13 05:29:34 UTC
To reproduce.
1. Start GDM greeter
2. Click on the accessibility icon; this brings up the accessibility dialog.
3. Close the dialog.  The user list / power buttons no longer have focus.  Pressing tab/arrow keys does not bring anything into focus.

The cause is the login window has the type hint GDK_WINDOW_TYPE_HINT_DOCK which is causing metacity to not give it focus.  This hint is not required as the login window explicity requests no decorations, to be skipped in the taskbar and pager, kept in the background etc.
Comment 1 Robert Ancell 2009-10-13 05:32:33 UTC
Created attachment 145326 [details] [review]
Removed the DOCK hint from the login window
Comment 2 Ray Strode [halfline] 2009-10-13 21:00:48 UTC
Review of attachment 145326 [details] [review]:

It seems like you may have found the root cause for bug 592183.  When we fix this bug we should consider reverting the commit from that bug.

::: gui/simple-greeter/gdm-greeter-login-window.c
@@ -1774,3 @@
         gtk_window_set_skip_pager_hint (GTK_WINDOW (login_window), TRUE);
         gtk_window_stick (GTK_WINDOW (login_window));
-        gtk_window_set_type_hint (GTK_WINDOW (login_window), GDK_WINDOW_TYPE_HINT_DOCK);

GDK_WINDOW_TYPE_HINT_DOCK was added to work around a metacity bug.  See bug 587732.

If we do revert the TYPE_HINT_DOCK change we should find some other workaround or fix for bug 587732 if it's still applicable.
Comment 3 Takao Fujiwara 2009-10-15 04:50:32 UTC
I think this problem can be fixed if we remove GDK_WINDOW_TYPE_HINT_DOCK with
the attachment 145326 [details] [review] , and I agree we need to revert the fix bug 592183.

I'm not sure how to reproduce bug #587732.
Comment 4 Robert Ancell 2009-10-15 05:02:37 UTC
I'm not seeing bug 587732 occuring in metacity 2.28.0
Comment 5 Brian Cameron 2009-10-16 02:45:19 UTC
I reverted the fix for bug #592183 and tried out the patch in comment #2.  

This does fix the focus problem.  Now after exiting the language dialog or the a11y dialog, focus remains in the login window.

However, I do see bug #592183 reappear and I do see the ""These windows do not support "save current setup" and will have to be restarted manually next time you log in", referring to Class: Login Window, Window: Gdm-simple-greeter." dialog.
Comment 6 Brian Cameron 2009-10-16 02:55:28 UTC
I re-opened bug #587732, since we need to reconsider fixing this issue in metacity.
Comment 7 Brian Cameron 2009-10-16 05:22:39 UTC
I mean "However, I do see bug #587732 reappear" in comment #5.  Sorry I repeated #592183 incorrectly there.
Comment 8 Takao Fujiwara 2009-11-02 08:59:51 UTC
I have no idea to reproduce bug #587732. I'm not sure if I need to clean up $HOME/.gconf for gdm user.
Unfortunately I don't see any error dialogs. 
However If you need, I can revert the fix of bug 592183.
Comment 9 Ray Strode [halfline] 2010-01-27 17:09:12 UTC

*** This bug has been marked as a duplicate of bug 59218 ***
Comment 10 Brian Cameron 2010-02-01 18:53:41 UTC

*** This bug has been marked as a duplicate of bug 592183 ***