-
Notifications
You must be signed in to change notification settings - Fork 377
updated landmark design pattern #3216
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
base: main
Are you sure you want to change the base?
Conversation
Hi Howard, I updated the URLs for landmark information in the index and coverage scripts. |
Hey @jongund, the generated link failed to build because of an unexpected structure, with the following message: "Error: Expected pattern landmarks to have an Example(s) section". This comes from wai-aria-practices' We could probably replicate that structure with these landmark patterns but I assume this proposed change here is intentional in how it's structured. So I've submitted wai-aria-practices#382 to relax this constraint instead. |
The coverage report build step result matches exactly except for a newline being added at the end of the |
@jongund w3c/wai-aria-practices#382 has been merged now so the Landmarks page renders in the generated preview now. Just the coverage report build step issue exists. #3216 (comment) should still resolve this |
@mcking65 |
The ARIA Authoring Practices (APG) Task Force just discussed The full IRC log of that discussion<jugglinmike> Topic: PR 3216 - Landmark pattern changes<jugglinmike> github: https://github.com//pull/3216 <jugglinmike> Matt_King: I think we'll skip this for now because jongunderson isn't here today |
@mcking65 |
Thank you for working on this. Unfortunately, I don't think this approach fits the APG information architecture. Problem: The new pattern page proposed in this PR combines practices, pattern, and examples into one large page. The currently published version of the pattern page sticks to the pattern template. I don't think we should change that. It points people to the practices page for practices. The currently published practices page is consistent with other practices pages. It does not need major revision. The primary problem we need to solve is the current production example pages are inconsistent with other example pages. The current landmark example pages:
A discussion to have is what do we want an example page for a landmark to contain. Open questions:
|
figure/figcaption
elementPreview new Landmarks Pattern Page
Preview updated landmark practices page (mostly updated links into patterns page)
WAI Preview Link (Last built on Thu, 27 Mar 2025 13:06:14 GMT).