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)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
mozilla34
People
(Reporter: yzen, Assigned: yzen)
Details
Attachments
(1 file, 2 obsolete files)
4.23 KB,
patch
|
yzen
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•10 years ago
|
||
Still need to test on Android.
Attachment #8472072 -
Flags: review?(eitan)
Comment 2•10 years ago
|
||
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.
Assignee | ||
Comment 3•10 years ago
|
||
(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?
Comment 4•10 years ago
|
||
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.
Assignee | ||
Comment 5•10 years ago
|
||
Attachment #8472072 -
Attachment is obsolete: true
Attachment #8472072 -
Flags: review?(eitan)
Attachment #8472380 -
Flags: review?(eitan)
Comment 6•10 years ago
|
||
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+
Assignee | ||
Comment 7•10 years ago
|
||
Carrying over +1
Attachment #8472380 -
Attachment is obsolete: true
Attachment #8472474 -
Flags: review+
Assignee | ||
Comment 8•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/cd34fb602fc3
Comment 9•10 years ago
|
||
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.
Description
•