Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extended Descriptions: not recognized by Ace #80

Open
gregoriopellegrino opened this issue Sep 7, 2023 · 3 comments
Open

Extended Descriptions: not recognized by Ace #80

gregoriopellegrino opened this issue Sep 7, 2023 · 3 comments

Comments

@gregoriopellegrino
Copy link

Launching Ace on the EPUB Extended Descriptions, going to the Images tab, for no image results in an extended description.

This is perhaps due to the fact that Ace only checks for the presence of the aria-describedby attribute.

@marisademeglio
Copy link
Member

The two tests in this book (extdescription-010, extdescription-020) use different methods to reference the extended description, and neither would be picked up by Ace.

extdescription-010 uses aria-details to reference the extended description, which I think Ace does not use when calculating the information on the Images tab.

extdescription-020 links to the description with a standard hyperlink, but the relationship between the image and description is not marked up in any way, so no automatic checker could infer the intention there.

Do you think there is an issue with the content or with Ace?

@gregoriopellegrino
Copy link
Author

In my opinion:

  • extdescription-010 is correct, and perhaps you need to ask Ace to implement the control
  • for extdescription-020 I think we should include a programmatic way to identify the link as a link to the extended description

@marisademeglio
Copy link
Member

I am having doubts about these tests, 010 is worded in a way that it does not test support for aria-details at all. And 020 tests that an ordinary hyperlink goes to another page.

As for Ace supporting aria-details, that sounds like an interesting convo - as a relative of aria-describedby, it could make sense, though aria-details is supposed to be for more complex descriptions (e.g. the table in our example here) so it wouldn't fit in Ace's report well.

cc @rdeltour @danielweck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants