302937 - Expose "click" action on text and image leaves of element containing onclick

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

For onclick we should parallel what we do for links, we expose the link actions
on the leaf text and image content. This helps the screen reader handle the
links and actionable items.

In the future we may provide more actions via XML Events or some way of
describing actions, and we will want to expose those actions on the actual leaf
content as well.

Comment 1

[Aaron Leventhal]

Attachment: 1) Have nsLinkableAccessible handle onclicks (probably should rename to nsActionableAccessible or something but that would bloat patch) , 2) Initialize action immediately

Comment 2

[parente]

Comment on attachment 191207 [details] [diff] [review]
1) Have nsLinkableAccessible handle onclicks (probably should rename to nsActionableAccessible or something but that would bloat patch) , 2) Initialize action immediately

Should nsAccessible::getNumActions return NS_OK?

Comment 3

[Aaron Leventhal]

Comment on attachment 191207 [details] [diff] [review]
1) Have nsLinkableAccessible handle onclicks (probably should rename to nsActionableAccessible or something but that would bloat patch) , 2) Initialize action immediately

Yeah lets have it return 0 and NS_OK for nsAccessible::getNumActions

Comment 4

[Johnny Stenback (:jst)]

Comment on attachment 191207 [details] [diff] [review]
1) Have nsLinkableAccessible handle onclicks (probably should rename to nsActionableAccessible or something but that would bloat patch) , 2) Initialize action immediately

sr=jst

Comment 5

[Christian :Biesinger (don't email me, ping me on IRC)]

  if (!mIsLink)  // Also initializes private data members 

That comment seems wrong now...

Comment 6

[Aaron Leventhal]

Looks like this was checked in ages ago and I forgot to mark it fixed.