Closed Bug 1053021 Opened 10 years ago Closed 10 years ago

[AccessFu] Speed up gesture intervals. Enable a configurable responsiveness to user gestures.

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla34

People

(Reporter: yzen, Assigned: yzen)

Details

Attachments

(1 file, 2 obsolete files)

As suggested by Jamie, we need to investigate if we can decrease the time intervals set for various gestures to get the screen reader feedback sooner.

Also we need to add an ability to use the original (more time forgiving) intervals.
Attached patch 1053021 patch v1 (obsolete) — Splinter Review
Still need to test on Android.
Attachment #8472072 - Flags: review?(eitan)
Out of curiosity, are there any use cases for making the longer delays optional? Both VoiceOver and NVDA have fairly short delays (NVDA is 0.25 sec for everything and could possibly be even shorter) and I've never heard of this being a problem for users. Configurability is good in many cases, but if it will never be used, it serves no purpose.
(In reply to James Teh [:Jamie] from comment #2)
> Out of curiosity, are there any use cases for making the longer delays
> optional? Both VoiceOver and NVDA have fairly short delays (NVDA is 0.25 sec
> for everything and could possibly be even shorter) and I've never heard of
> this being a problem for users. Configurability is good in many cases, but
> if it will never be used, it serves no purpose.

The thinking was to, possibly, accommodate a wider screen reader user range, someone also with motor disability or similar. What do you think?
Whether we expose this option to users or not is a separate UX discussion. What we have in trunk currently works, and could possibly be useful for motor disabilities and limited dexterity users, so I think the option is worth preserving.
Attached patch 1053021 patch v2 (obsolete) — Splinter Review
Attachment #8472072 - Attachment is obsolete: true
Attachment #8472072 - Flags: review?(eitan)
Attachment #8472380 - Flags: review?(eitan)
Comment on attachment 8472380 [details] [diff] [review]
1053021 patch v2

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

How about having one constant to change?
So something like:
const TIMEOUT_MULTIPLIER = 1 // Multiply timeouts by this constant, x2 works great too for slower users.
Attachment #8472380 - Flags: review?(eitan) → review+
Attached patch 1053021 patch v3Splinter Review
Carrying over +1
Attachment #8472380 - Attachment is obsolete: true
Attachment #8472474 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/cd34fb602fc3
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: