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

monorepo: add html-aria history #2185

Closed
wants to merge 961 commits into from
Closed

Conversation

pkra
Copy link
Member

@pkra pkra commented May 20, 2024

  • add html-aria history
  • remove extraneous files
  • adjust links to aria-common
  • consider whether or not we should move html-aria right now

patrickhlauke and others added 30 commits May 16, 2021 14:09
… 1.1

Since this has been removed from the now renamed table, makes more sense to point to the spec
Before jumping straight into explaining what each column is, give some context first for the overall table.
minor source cleanup to try and force the build process again.
update results for `aria-required` from the html validator
html validator has updated implementation for svg.  changes to this element are no longer at risk
* HTML Validator has implemented rule for `aria-multiselectable` on `select` element
* HTML Validator has fixed exception to `dfn role=term` - now produces warning that `role=term` is not necessary on `dfn` element.
update the global aria-* attributes url from using the tr/wai-aria-1.1 url to instead use `data-cite=...` as with other ARIA references.
* HTML validator has implemented `placeholder`/`aria-placeholder` rule.
* HTML validator has implemented updated rules for `br` role allowances. (removes at risk)
* HTML validator has implemented updates role allowances for `figure` (removes at risk)
* HTML validator has updated rules for `input type=radio / checkbox` removing the minor exceptions.
updated html validator results
remove instances of 'must' from non-normative section
html validator has implemented rules for `aria-rowspan` and `aria-colspan`.  the associated rules are no longer at risk with 2 implementations.
scottaohara and others added 27 commits June 14, 2023 12:42
closes #474

changes ref of section element without name from 'no corresponding role' to `role=generic`.

other changes made in this PR:
- remove duplicate instances of code elements from ARIA roles (mostly instances of role=generic).
- clarification for section element allowed roles - role=region not recommended in general, not just if the element is named since using the role w/out naming would be useless.  mention authors SHOULD NOT use role=generic on the element, per the change from no corresponding role.
the s element was remapped from generic to role=deletion in html aam. 

the only potential change here for conformance checkers is to signal a warning that use of the deletion role on the s element is unnecessary.

this pr will close #466
* Update button and input button type allowances

closes #444
closes #395
closes #298

This PR makes the allowances for button-elements (button, input type=button|image|submit|reset) more consistent with each other.  Additionally, slider and gridcell are now listed under the allowed roles for these elements.

the LOE to properly create the necessary UX for some of these roles when specified, especially on the `input` type buttons, is rather large _but_ possible.  Any failures to not implement these properly would be caught by other WCAG rules and likely other automated checks.

* add missing slider allowance for input type=image
* add changelog
* add allowed role for aside element - related to, w3c/html-aria#411 but does not resolve the issue.
* update change log
(In this instance, it doesn’t seem needed.)
* Editorial: revise redundant role declaration example closes #478
* remove errant `;` closes #479
merging with 1.5 reflections in checkers, and the fact this is an update to match HTML AAM.


Update: naming prohibited changes for hgroup, address, rp
hgroup, address have each been mapped to the group role, and thus no longer prohibit naming.
rp is not an element that _should_ be exposed to users, and even when it is - if ruby content is not supported - then it really should not be given a name then, either.
Some aria-hidden links refer to WAI-ARIA 1.1, not 1.2.

Co-authored-by: Scott O'Hara <scottaohara@users.noreply.github.com>
Clarify and consolidate img ARIA allowances

This PR closes #424 and #452

* clarify when img role is allowed
* add meter to allowed role for img
* similar to progressbar, meter should thus also be allowed.
* fix img lacking alt allowance
* update changelog
update some missing aria attributes for the specified roles, unrelated to the overall img role tests
closes #488 

allows role=separator on elements exposed as a button.  

any other author requirements for appropriately setting that up / ensuring the correct information is communicated would still need to be evaluated, but separate from whether the role is allowed or not.

* Addition: allow role=separator on button elements
* update changelog
moves td and th elements into the correct alphabetical order in the elements table.
…ibute usage by HTML element" table (header element) (#502)

Fixes typo / link for allowed implicit role
---------

Co-authored-by: Scott O'Hara <scottaohara@users.noreply.github.com>
…s for "form" element (#500)

Co-authored-by: Scott O'Hara <scottaohara@users.noreply.github.com>
Editorial: input type=image should match type=reset|submit

The allowed roles were made consistent in an earlier PR, but the other accompanying 'not recommended' guidance was missed.

closes #504

expand on the 'if possible' paragraph to call out that it'd be better for authors to use the button element instead of these legacy (though still valid) input button types.
Previous: "... if the parent list item does not ...."

Corrected text: "... if the parent list element does not ..."
* Correction: add missing allowance for aria-* on meter element

closes #517

adds missing allowance for any aria-* attributes applicable to the meter role.
* remove sticky column headers per style breaking

* Update makeup.css

don't know why the last publish failed.  maybe if i remove the css instead of commenting it out?  this is silly..
Copy link

netlify bot commented May 21, 2024

Deploy Preview for ephemeral-daifuku-0dc7d7 ready!

Name Link
🔨 Latest commit 336eacd
🔍 Latest deploy log https://app.netlify.com/sites/ephemeral-daifuku-0dc7d7/deploys/664ce7ce0119f900082bc799
😎 Deploy Preview https://deploy-preview-2185--ephemeral-daifuku-0dc7d7.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@spectranaut
Copy link
Contributor

Do we actually own this document, yet?

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

Successfully merging this pull request may close these issues.

None yet