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 684730 - 'log in as different user' not read well by orca at all
'log in as different user' not read well by orca at all
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: login-screen
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Ray Strode [halfline]
gnome-shell-maint
3.6.1
Depends on:
Blocks:
 
 
Reported: 2012-09-24 17:00 UTC by Matthias Clasen
Modified: 2012-10-12 14:32 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
loginDialog: Use the same focus root for dialog and ctrl-alt-tab (1.48 KB, patch)
2012-10-10 20:44 UTC, Florian Müllner
committed Details | Review
loginDialog: Add focus indication (1.51 KB, patch)
2012-10-10 20:45 UTC, Florian Müllner
committed Details | Review

Description Matthias Clasen 2012-09-24 17:00:58 UTC
When you click on this (you can't keynav to it, which is a different issue), you get to a screen with a username entry. Orca reads that as just 'text'. Then, if you focus to the buttons, it reads them as just 'push button'.
Comment 1 Matthias Clasen 2012-09-24 17:22:24 UTC
actually, you can't even focus the buttons...
Comment 2 Florian Müllner 2012-10-10 20:44:38 UTC
Created attachment 226204 [details] [review]
loginDialog: Use the same focus root for dialog and ctrl-alt-tab

Adding a group to the Ctrl-Alt-Tab popup will also add it to the
focus manager. Due to that, we currently end up with two focus
groups added for the login dialog - an explicit one for the entire
dialog, and an implicit one for the main content group.
When doing keynav, we ascend in the widget hierarchy from the
currently focused actor until we find a valid focus root, so
adding a children of the dialog as focus root breaks keynav to
any actors that are not inside the main content group.
The simple fix is to use the same group in both cases.
Comment 3 Florian Müllner 2012-10-10 20:45:18 UTC
Created attachment 226205 [details] [review]
loginDialog: Add focus indication

While keynav works on login screen and unlock dialog, most elements
currently miss a focus indication.
Comment 4 Florian Müllner 2012-10-10 20:50:22 UTC
(In reply to comment #0)
> When you click on this (you can't keynav to it, which is a different issue),
> you get to a screen with a username entry. Orca reads that as just 'text'.
> Then, if you focus to the buttons, it reads them as just 'push button'.

With the attached patches (well, only the first actually), I can keynav to any element on the login screen. I cannot reproduce the issue with missing a11y-labels though - when I activate Orca, everything read makes some sense to me ...

(tested with current master + patches in bug 685201 + patches in this bug)
Comment 5 Matthias Clasen 2012-10-11 03:03:24 UTC
Works fine in my testing as well.
Comment 6 Florian Müllner 2012-10-11 16:55:14 UTC
Can I get review on this? Both patches are fairly straightforward.
Comment 7 Jasper St. Pierre (not reading bugmail) 2012-10-11 16:56:11 UTC
Review of attachment 226204 [details] [review]:

This looks like a simple oversight. Easy enough.
Comment 8 Jasper St. Pierre (not reading bugmail) 2012-10-11 16:57:50 UTC
Review of attachment 226205 [details] [review]:

This too.
Comment 9 Florian Müllner 2012-10-11 17:06:14 UTC
Attachment 226204 [details] pushed as 3abfcda - loginDialog: Use the same focus root for dialog and ctrl-alt-tab
Attachment 226205 [details] pushed as 8cc54ce - loginDialog: Add focus indication

(In reply to comment #5)
> Works fine in my testing as well.

OK, so let's close this for now ...
Comment 10 Ray Strode [halfline] 2012-10-12 14:32:02 UTC
awesome