When screen reader is toggled off the a11y panel disappears

RESOLVED FIXED

Status

Firefox OS
Gaia::System
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: eeejay, Unassigned)

Tracking

(Blocks: 1 bug, {access})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

4 years ago
STR:

1. Enable a11y panel in settings
2. Toggle screen reader on via panel.
3. Toggle screen reader off.

Results: a11y panel goes away.
Expected: a11y panel should remain once it is enabled.
(Reporter)

Comment 1

4 years ago
Created attachment 8469613 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/22646
Attachment #8469613 - Flags: review?(alive)
(Reporter)

Comment 2

4 years ago
Created attachment 8469616 [details]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/22646#issuecomment-51537051

This is how I think the behavior should be. I think the previous patch misunderstood the purpose for this, which is to show the panel permanently once the user starts toying with the screen reader.
Attachment #8469616 - Flags: review?(alive)
Attachment #8469616 - Flags: feedback?(yzenevich)

Comment 3

4 years ago
This use case you provided makes sense.

I think there should be 2 type of settings in accessibility.js One type is responsible for showing/hiding a11y panel through Show Accessibility settings ON/OFF. Another one is responsible for switching accessibility.screenreader setting ON/OFF. Basically whenever one of those settings is ON then the a11y panel should be present. But ONLY when Show Accessibility settings are set to OFF then we hide the a11y panel.

What do you think about this?
Comment on attachment 8469616 [details]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/22646#issuecomment-51537051

I like this.
Attachment #8469616 - Flags: feedback?(yzenevich) → feedback+
(Reporter)

Updated

4 years ago
Attachment #8469616 - Attachment is obsolete: true
Attachment #8469616 - Flags: review?(alive)
(Reporter)

Comment 5

4 years ago
(In reply to Alexey Novak from comment #3)
> This use case you provided makes sense.
> 
> I think there should be 2 type of settings in accessibility.js One type is
> responsible for showing/hiding a11y panel through Show Accessibility
> settings ON/OFF. Another one is responsible for switching
> accessibility.screenreader setting ON/OFF. Basically whenever one of those
> settings is ON then the a11y panel should be present. But ONLY when Show
> Accessibility settings are set to OFF then we hide the a11y panel.
> 
> What do you think about this?

I think the patch I submitted has a similar effect and is pretty straight forward. If the screen reader is activated (no matter how), make sure the panel is showing, and don't hide it again unless the user goes into dev settings and disables it.
Attachment #8469613 - Flags: review?(alive) → review+
https://github.com/mozilla-b2g/gaia/commit/9062a9a2de50d56ddba733d23b90993d3a276457
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.