Closed Bug 1182208 Opened 7 years ago Closed 7 years ago

[AccessFu][Android] enabled support for range slider control.

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed

People

(Reporter: yzen, Assigned: eeejay)

References

Details

(Keywords: access)

Attachments

(1 file)

At the moment adjusting values for input range is not working on android. We need to figure out what action/event needs to happen when we want to increase or decrease range input's value.
Duplicate of this bug: 1182219
Attachment #8650143 - Flags: review?(yzenevich)
Attachment #8650143 - Flags: review?(mark.finkle)
Comment on attachment 8650143 [details] [diff] [review]
Add support for android scrolling and range accessibility actions.

Seems fine to me.
Attachment #8650143 - Flags: review?(mark.finkle) → review+
Comment on attachment 8650143 [details] [diff] [review]
Add support for android scrolling and range accessibility actions.

Review of attachment 8650143 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, just one comment to address

::: accessible/jsat/AccessFu.jsm
@@ +134,5 @@
>      Services.obs.addObserver(this, 'Accessibility:Focus', false);
>      Services.obs.addObserver(this, 'Accessibility:ActivateObject', false);
>      Services.obs.addObserver(this, 'Accessibility:LongPress', false);
> +    Services.obs.addObserver(this, 'Accessibility:ScrollForward', false);
> +    Services.obs.addObserver(this, 'Accessibility:ScrollBackward', false);

These observers also need to removed in _disable.
Attachment #8650143 - Flags: review?(yzenevich) → review+
Assignee: nobody → eitan
https://hg.mozilla.org/mozilla-central/rev/581329d6602b
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Summary: [AccessFu][Androd] enabled support for range slider control. → [AccessFu][Android] enabled support for range slider control.
You need to log in before you can comment on or make changes to this bug.