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 736821 - a11y: Most of the accessible tree is not exposed, and is partially wrong
a11y: Most of the accessible tree is not exposed, and is partially wrong
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
3.13.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2014-09-17 17:03 UTC by Alejandro Piñeiro Iglesias (IRC: infapi00)
Modified: 2014-09-18 13:39 UTC
See Also:
GNOME target: 3.14
GNOME version: ---


Attachments
a11y: Set again NO_AT_BRIDGE=1 before calling meta_init (1010 bytes, patch)
2014-09-17 17:06 UTC, Alejandro Piñeiro Iglesias (IRC: infapi00)
committed Details | Review

Description Alejandro Piñeiro Iglesias (IRC: infapi00) 2014-09-17 17:03:44 UTC
STEPS TO REPRODUCE IT

Open accerciser with gnome-shell 3.13.92. Under gnome-shell, you should see a tree with a lot of elements. They are just three, and two are of role window. On clutter, right now, only the stage uses that role. That would imply that gnome-shell is using two clutter stages, but that is not the case.

Due this, although some events, like focus changes, are properly emitted, and exposed by Orca, other like selection changes are not due the missing accessibles. That makes alt+tab and ctrl+alt+tab fairly totally inaccessible. And probably there are other features not accessible.

NOTES

This is a collateral effect of bug 735908. After the change I also removed setting/unsetting NO_AT_BRIDGE around meta_init, assuming that they would not be needed anymore. So that part of the patch was just cleaning. The fact that everything was working after removing them seemed to confirm my theory. Two weeks later, while testing 3.13.92 just in case, I found this bug, so obviously I was wrong.

Patch is coming.
Comment 1 Alejandro Piñeiro Iglesias (IRC: infapi00) 2014-09-17 17:06:20 UTC
Created attachment 286409 [details] [review]
a11y: Set again NO_AT_BRIDGE=1 before calling meta_init

This prevents the call of gtk_init_xx to load the bridge, that affects the bridge being loaded.

All this would not be required if a proper API is implemented as mentioned on bug 735908. Due the freezes, that would be postponed to next cycle.
Comment 2 Alejandro Piñeiro Iglesias (IRC: infapi00) 2014-09-17 17:10:08 UTC
Setting target, as right now the accessibility support is mostly broken.

The patch is just re-adding two lines that were on gnome-shell during years, so I assume that it would be easy to get it approved. If it is approved I will send the freeze break request.
Comment 3 Jasper St. Pierre (not reading bugmail) 2014-09-17 17:11:03 UTC
Review of attachment 286409 [details] [review]:

Fine by me.
Comment 4 André Klapper 2014-09-17 20:04:05 UTC
(In reply to comment #1)
> All this would not be required if a proper API is implemented as mentioned on
> bug 735908.

Not sure if I can follow, but: Is there a dedicated ticket to track the request for a proper API (as bug 735908 is closed)?

And r-t approval one of two here already for the patch in comment 3.
Comment 5 Alejandro Piñeiro Iglesias (IRC: infapi00) 2014-09-18 10:28:22 UTC
(In reply to comment #4)
> (In reply to comment #1)
> > All this would not be required if a proper API is implemented as mentioned on
> > bug 735908.
> 
> Not sure if I can follow, but: Is there a dedicated ticket to track the request
> for a proper API (as bug 735908 is closed)?

What I mean is that eventually, a new API would allow to avoid the need to set that envvar, and to force setting the AtkUtil as bug 735908 needed to do. There is not still a ticket for that new API. That work would be postponed for the next cycle, and probably will just open it when I have something to show.

> And r-t approval one of two here already for the patch in comment 3.

Ok, thanks.
Comment 6 Alejandro Piñeiro Iglesias (IRC: infapi00) 2014-09-18 13:39:07 UTC
(In reply to comment #3)
> Review of attachment 286409 [details] [review]:
> 
> Fine by me.

Freeze break request accepted:
https://mail.gnome.org/archives/release-team/2014-September/msg00080.html

Patch committed. Closing bug.