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

Mismatch in the accessibility references in the current text and schema... #216

Closed
iherman opened this issue Jun 11, 2018 · 12 comments
Closed

Comments

@iherman
Copy link
Member

iherman commented Jun 11, 2018

The current draft says:

The infoset SHOULD include a link to an accessibility report when one is available for a Web Publication. It is RECOMMENDED that the report be included as a resource of the Web Publication.

which suggests that the acceessiblity report is more a structural property (i.e., linking out to a full report) rather than descriptive. This is in contrast with the discussion in Toronto where accessibility was listed as a descriptive property

Cc @avneeshsingh @GeorgeKerscher

@iherman
Copy link
Member Author

iherman commented Jun 11, 2018

My proposal would be to allow for both: recommend the usage of the accessibility terms of schema.org but, also introduce a structural property for an accessibility report.

@laudrain
Copy link

Structural properties are supposed to "express key structures" (https://www.w3.org/TR/wpub/#infoset-expl).
Personally, I don't see accessibility metadata expressed in a report to be "key structures".
But more as giving global information on the accessibility state of the publication.

@iherman
Copy link
Member Author

iherman commented Jun 11, 2018

@laudrain: the properties described in schema.org (see also the relevant wiki page) may be enough then, at least in the core manifest. If so, the description of the infoset item may have to be changed...

@laudrain
Copy link

@iherman: you mean that "linking out to a full report" implies "structural property" ?
But it could also be "extensibility" as links to other "metadata records" are in 3.5 Extensibility.
Then the proposal could become:

  • a11y properties in schema.org in 3.3 Descriptive properties
  • link to the full a11y report in 3.5 Extensibility
    What do @avneeshsingh @GeorgeKerscher think?

@iherman
Copy link
Member Author

iherman commented Jun 11, 2018

Yes, that is an alternative, too.

@GeorgeKerscher
Copy link

GeorgeKerscher commented Jun 16, 2018 via email

@iherman
Copy link
Member Author

iherman commented Jun 17, 2018

@GeorgeKerscher this may then be another example for the separate 'list' discussed in #225. Which is perfectly fine, it only requires to define another 'rel' value to identify that this is a report, and the rest falls out painlessly...

@avneeshsingh
Copy link

The accessibility metadata is mostly for discovery. The external report provides felexibility beyond schema.org accessibility metadata. It can also have ONIX codes in it.

I think that we are discussing the list of resources and not resources in default reading order. So, does the recommendation to include accessibility report in list of resources create any major problem?

@iherman
Copy link
Member Author

iherman commented Jun 18, 2018

I think that we are discussing the list of resources and not resources in default reading order. So, does the recommendation to include accessibility report in list of resources create any major problem?

No:-) The only reason I raised this is when mapping the infoset onto the manifest, we end up with two different things:

  1. usage of the schema.org accessibility terms that you know very well
  2. the possibility to add link to a separate accessibility report

And my only question was whether it is all right to have both. I have the impression that we are fine having both indeed...

@avneeshsingh
Copy link

Indeed.

@clapierre
Copy link

Yes we would want to have both, as ONIX currently doesn't support all of the new accessibility metadata for conformance and discovery but does allow for a link to a report which would include anything it currently does not support, so having both would be ideal.

@iherman
Copy link
Member Author

iherman commented Jun 18, 2018

Changing this to editorial, I think we have an agreement

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

No branches or pull requests

5 participants