Closed
Bug 752131
Opened 12 years ago
Closed 12 years ago
[AccessFu] Do not return empty string elements in utterances
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: eeejay, Assigned: eeejay)
Details
Attachments
(1 file)
3.62 KB,
patch
|
davidb
:
review+
|
Details | Diff | Splinter Review |
Originally I chose to not filter out empty strings from utterances. With the thought that ATs could have some understanding about how the utterance is constructed and modify their presentation. For example on a focus event, strings in odd indexes are names or values, strings in even indexes are roles or descriptions. The problem is that it seems like TalkBack delimits each element from the utterance with spaces, so if you land on plain text, and the utterance (in Android it is AccessibilityEvent.getText()) would be ['', 'some text']. This would be sent to the TTS as ' some text'. the preceding whitespace delays the speech, so users get a laggy experience (well a more laggy experience :)
Assignee | ||
Comment 1•12 years ago
|
||
This should do it. Nothing too fancy. It feels liberating not to have to return two elements each time. I might suggest later on not clumping roles and states into the previous "description" element and to have them seperate. This introduces interesting questions about the grammar boundaries, and how things should be localized.
Attachment #621234 -
Flags: review?(dbolter)
Comment 2•12 years ago
|
||
I want to go over some high level stuff with you before review (probably Monday).
Updated•12 years ago
|
Attachment #621234 -
Flags: review?(dbolter) → review+
Assignee | ||
Comment 3•12 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/53face03b63f
Comment 4•12 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/53face03b63f
Assignee: nobody → eitan
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
You need to log in
before you can comment on or make changes to this bug.
Description
•