Note: There are a few cases of duplicates in user autocompletion which are being worked on.

[AccessFu] Provide feedback when entering an entry field for editing

RESOLVED FIXED in mozilla16

Status

()

Core
Disability Access APIs
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: MarcoZ, Assigned: eeejay)

Tracking

Trunk
mozilla16
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
STR:
1. Open a page that has an entry field.
2. navigate to it.
3. Press Enter to focus on it so one can type.

Expected: One should get feedback that one can now type.
Actual: No feedback is given.
(Assignee)

Updated

5 years ago
Assignee: nobody → eitan
(Assignee)

Comment 1

5 years ago
Do we only need feedback when we enter editing mode, or when we exit as well? I have a WIP, and the exit message gets clobbered by the focus move utterance. It sounds self evident that we are now navigating, so I don't see this too much as an issue.

I guess if we add a key to exit editing without arrowing out of it, it should announce something.

Thoughts?
(Reporter)

Comment 2

5 years ago
I agree on both counts. :)
(Assignee)

Comment 3

5 years ago
Created attachment 639413 [details] [diff] [review]
Announce "editing" when in editing mode. Use a11y states and focus to determine it.
Attachment #639413 - Flags: review?(dbolter)
Comment on attachment 639413 [details] [diff] [review]
Announce "editing" when in editing mode. Use a11y states and focus to determine it.

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

::: accessible/src/jsat/AccessFu.jsm
@@ +368,5 @@
> +
> +        let [,extState] = Utils.getStates(acc);
> +        let editableState = extState &
> +          (Ci.nsIAccessibleStates.EXT_STATE_EDITABLE |
> +           Ci.nsIAccessibleStates.EXT_STATE_MULTI_LINE);

I'm unsure if this captures DesignMode/content editable... might want to // XXX that but I'm also not sure we'll care about it soon. Your call.

::: accessible/src/jsat/Presenters.jsm
@@ +358,5 @@
> +  editingModeChanged: function AndroidPresenter_editingModeChanged(aIsEditing) {
> +    this._appAnnounce(UtteranceGenerator.genForEditingMode(aIsEditing));
> +  },
> +
> +  _appAnnounce: function _appAnnounce(aUtterance) {

Just curious why you named it "appAnnounce"? Is it because this is where we are reminded we are an app(lication) sending an a11y event on android?
Attachment #639413 - Flags: review?(dbolter) → review+
(Assignee)

Comment 5

5 years ago
(In reply to David Bolter [:davidb] from comment #4)
> Comment on attachment 639413 [details] [diff] [review]
> Announce "editing" when in editing mode. Use a11y states and focus to
> determine it.
> 
> Review of attachment 639413 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: accessible/src/jsat/AccessFu.jsm
> @@ +368,5 @@
> > +
> > +        let [,extState] = Utils.getStates(acc);
> > +        let editableState = extState &
> > +          (Ci.nsIAccessibleStates.EXT_STATE_EDITABLE |
> > +           Ci.nsIAccessibleStates.EXT_STATE_MULTI_LINE);
> 
> I'm unsure if this captures DesignMode/content editable... might want to //
> XXX that but I'm also not sure we'll care about it soon. Your call.
> 

Yes, unlike the previous DOM solution, this should account for design mode.

> ::: accessible/src/jsat/Presenters.jsm
> @@ +358,5 @@
> > +  editingModeChanged: function AndroidPresenter_editingModeChanged(aIsEditing) {
> > +    this._appAnnounce(UtteranceGenerator.genForEditingMode(aIsEditing));
> > +  },
> > +
> > +  _appAnnounce: function _appAnnounce(aUtterance) {
> 
> Just curious why you named it "appAnnounce"? Is it because this is where we
> are reminded we are an app(lication) sending an a11y event on android?

An inspiration from JellyBean API where they have an accessibility event just for things like this.
(Assignee)

Comment 6

5 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/d540e12b6c6b
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/d540e12b6c6b
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.