Adobe Reader: Prefer name over text content for buttons #3640

Closed
nvaccessAuto opened this Issue Nov 12, 2013 · 4 comments

2 participants

@nvaccessAuto

Reported by jteh on 2013-11-12 06:15
Issue raised by Adobe.

For some buttons, IPDDomNode::GetTextContent returns something different to the name. GetTextContent is supposed to account for alternate and actual text, but it seems that overriding a button's name is not considered alternate/actual text. Apparently, authors can override the name using the tooltip and this is what they (and Adobe) expect an AT user to see.

I guess it makes sense to do this for all controls for which the name is generally the content; i.e. the useNameAsContent flag in the backend.

@nvaccessAuto

Comment 2 by James Teh <jamie@... on 2013-11-13 03:12
In [d196b3f]:
```CommitTicketReference repository="" revision="d196b3f759b4a616ddd9833bdbe360ccece7223e"
In browse mode in Adobe Reader, the correct text is now rendered for buttons, etc. where the label has been overridden using a tooltip or other means.

In the backend, where useNameAsContent is true, IPDDomNode::GetName is now tried before IPDDomNode::GetTextContent.
Re #3640.

@nvaccessAuto

Comment 3 by James Teh <jamie@... on 2013-11-13 03:13
In [d6ad6ba]:
```CommitTicketReference repository="" revision="d6ad6ba8bf2f68251dd980e9350a5521f4e47069"
Merge branch 't3640' into next

Incubates #3640.

Changes:
Added labels: incubating
@nvaccessAuto

Comment 6 by James Teh <jamie@... on 2013-11-26 22:59
In [0b55f19]:
```CommitTicketReference repository="" revision="0b55f195afa58032540ad10a07a771dd526fa915"
In browse mode in Adobe Reader, the correct text is now rendered for buttons, etc. where the label has been overridden using a tooltip or other means.

In the backend, where useNameAsContent is true, IPDDomNode::GetName is now tried before IPDDomNode::GetTextContent.
Fixes #3640.

Changes:
Removed labels: incubating
State: closed
@nvaccessAuto

Comment 7 by jteh on 2013-11-26 23:04
Changes:
Milestone changed from next to 2014.1

@jcsteh jcsteh was assigned by nvaccessAuto Nov 10, 2015
@nvaccessAuto nvaccessAuto added this to the 2014.1 milestone Nov 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment