Closed Bug 1218897 Opened 6 years ago Closed 6 years ago

[Accessibility] Screen reader can not access activity window.

Categories

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: yzen, Assigned: etienne)

References

Details

(Keywords: access, regression)

Attachments

(1 file)

This seems like a regression because we used to access activity windows with the screen reader.

The core issue here is that activity window is rendered inside an app window container that is marked inactive and thus has aria-hidden attribute set to "true".

Steps to reproduced were reported by Julien in https://bugzilla.mozilla.org/show_bug.cgi?id=1211115#c12 

1. Open the Messages app
2. Activate the "new message" button.
3. Activate the "+" button.

=> Then I couldn't select a contact. The whole activity window is not visible to the screen reader.
See Also: → 1211115
QA Contact: jthomas
Caused by changes made in Bug 1144132

B2g Inbound Regression Window

Last Working
Environmental Variables:
Device: Flame 2.5
BuildID: 20150622170244
Gaia: c4d3b8fd78bcd7eff1b127060ec6490a891b7a35
Gecko: 2ab4034b2cc1
Version: 41.0a1 (2.5) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:41.0) Gecko/41.0 Firefox/41.0

First Broken
Environmental Variables:
Device: Flame 2.5
BuildID: 20150623190325
Gaia: eb0d4aefa62b20420d6fa0642515a110daca5d97
Gecko: 5a3ee6baf143
Version: 41.0a1 (2.5) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:41.0) Gecko/41.0 Firefox/41.0


Last Working gaia / First Broken gecko - This issue does NOT occur with broken Gecko
Gaia: c4d3b8fd78bcd7eff1b127060ec6490a891b7a35
Gecko: 5a3ee6baf143

Last Working gecko / First Broken gaia - This issue DOES occur with broken Gaia
Gecko: 2ab4034b2cc1
Gaia: eb0d4aefa62b20420d6fa0642515a110daca5d97

B2G Inbound Pushlog:
https://github.com/mozilla-b2g/gaia/compare/c4d3b8fd78bcd7eff1b127060ec6490a891b7a35...eb0d4aefa62b20420d6fa0642515a110daca5d97
Blocks: 1144132
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(jmercado)
Gabriele you handled another issue caused by Bug 1144132.  Can you weigh in here as well?
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmercado) → needinfo?(gsvelto)
This is most likely due to the window management changes which I'm not very familiar with. I only handled the gecko part of bug 1144132 so I'm NI'ing Etienne who certainly knows more.
Flags: needinfo?(gsvelto) → needinfo?(etienne)
(In reply to Gabriele Svelto [:gsvelto] from comment #3)
> This is most likely due to the window management changes

Yep.
Assignee: nobody → etienne
Flags: needinfo?(etienne)
Attachment #8680609 - Flags: review?(apastor)
Comment on attachment 8680609 [details] [review]
[gaia] etiennesegonzac:bug-1218897 > mozilla-b2g:master

LGTM, thanks!
Attachment #8680609 - Flags: review?(apastor) → review+
https://github.com/mozilla-b2g/gaia/commit/1a7ed5c8c18f1765a39c65fe19e6ec6796c86a8b
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.