Implementation section updated #682
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
GeorgeKerscher
requested review from
mattgarrish,
gregoriopellegrino,
clapierre,
madeleinerothberg,
avneeshsingh,
gautierchomel and
chrisONIX
gregoriopellegrino
approved these changes
Apr 1, 2025
mattgarrish
reviewed
Apr 1, 2025
|
|
||
| <p>These guidelines provide a general framework and make | ||
| suggestions on the display of accessibility | ||
| metadata. It is not a normative description of what must be | ||
| done. It is instructive to provide examples | ||
| of implementations from the community. </p> | ||
|
|
||
| <p>Linked below are static pages that show real-life | ||
| implementations. We have captured these examples from | ||
| <p>We have created a webpage that captures examples from |
There was a problem hiding this comment.
Best to avoid pronouns and be specific. I'd probably change this to:
The Publishing Community Group has created a web page that captures examples from web sites belonging to organizations that have agreed to showcase the the work they have done to display accessibility metadata.
mattgarrish
reviewed
Apr 1, 2025
Comment on lines
2501
to
2506
| <li>29-Mar-2025: Fixed language describing accessible rich content. See <a | ||
| href="https://github.com/w3c/publ-a11y/issues/671">issue 671</a>.</li> | ||
| <li>29-Mar-2025: Fixed example 12 that was using incorrect strings for filtering. See <a | ||
| href="https://github.com/w3c/publ-a11y/issues/673">issue 673</a>.</li> | ||
| <li>29-Mar-2025: Revised Implementation section to point to an external page rather than embedding static images of examples.</li> | ||
| </ul> |
There was a problem hiding this comment.
For the change logs, it's always newest entries first, so the entries you added should be moved to the top.
Merge branch 'main' into implementation-section
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Two commits here. One is for the implementation sectioion. I commented out the link to the issue requesting companies volunteer implementations. I thought this would be reused on those pages.
I have the link to the implementation page, but still need to add the actual URL.
Made three entries to the change log.