-
Notifications
You must be signed in to change notification settings - Fork 796
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
chore(release): 4.3.4 #3218
chore(release): 4.3.4 #3218
Conversation
…3136) * tests: ensure after and reporter methods can run in isolated context * Update test/integration/full/isolated-env/isolated-env.js Co-authored-by: Wilco Fiers <[email protected]> * cleanup Co-authored-by: Wilco Fiers <[email protected]>
* Fix nativeSelectValue not picking up selection * Use props instead * Add test for selected prop
* add visible checks and tests * fix typo * enable screenreader on isVisible * add tests * revert playground * add aria-hidden tests * add test and clean up * fix test to pass if aria-invalid is false on hidden element
As per the spec, the context parameter can be passed one of the following: > A `NodeList` such as returned by `document.querySelectorAll`. Source: https://www.deque.com/axe/core-documentation/api-documentation/#context-parameter
* fix(is-visible): do not error if window.Node does not exist * circle * Update test/test-virtual-rules.js Co-authored-by: Wilco Fiers <[email protected]> Co-authored-by: Wilco Fiers <[email protected]>
… messageKey (#3181) * tests: check that every check has a message for every return type and messageKey * finalize * fixes * typo
…nding size (#3186) * fix(color-contrast): account for elements that do not fill entire bounding size * not private
* feat: create issue impact documentation * Update doc/issue_impact.md Co-authored-by: Wilco Fiers <[email protected]> * Update issue_impact.md * Update doc/issue_impact.md Co-authored-by: Stephen Mathieson <[email protected]> Co-authored-by: Wilco Fiers <[email protected]> Co-authored-by: Stephen Mathieson <[email protected]>
…istbox (#3195) * fix(aria-allowed-children,aria-allowed-parent): allow group role in listbox * fix tests
* fix(color-contrast): properly blend multiple alpha colors * revert shadow * add back shadow color flatten * fix * fix ie11 * type * typo
* fix: Separate Level AAA rules from A and best-practices * Don't log use of wcag21aaa as a tag * address comments
* Update rule-proposal.md * Update doc/rule-proposal.md Co-authored-by: Stephen Mathieson <[email protected]> Co-authored-by: Stephen Mathieson <[email protected]>
* fixes * finalize * sort * test * fix tests * Update lib/commons/color/get-background-stack.js Co-authored-by: Wilco Fiers <[email protected]> * fixes * fix(color-contrast): correctly apply page background color Co-authored-by: Wilco Fiers <[email protected]>
* fix: greater consistency of help / description text * Apply suggestions from code review Co-authored-by: Steven Lambert <[email protected]> * Update lib/rules/p-as-heading.json Co-authored-by: Glenda Sims <[email protected]> Co-authored-by: Steven Lambert <[email protected]> Co-authored-by: Glenda Sims <[email protected]>
* fix(color-contrast): correctly handle nested scroll regions * revert changes from previous pr
…z-index (#3209) * fix(color-contrast): correctly work with positioned elements without z-index * correct link * add back test * fix * revert playground * typo * typo * Update lib/commons/dom/get-rect-stack.js Co-authored-by: Wilco Fiers <[email protected]> * Update test/commons/dom/get-element-stack.js Co-authored-by: Wilco Fiers <[email protected]> * fix Co-authored-by: Wilco Fiers <[email protected]>
…ildren (#3203) * fixes * finalize * sort * test * fix tests * Update lib/commons/color/get-background-stack.js Co-authored-by: Wilco Fiers <[email protected]> * fixes * split out pr * order * rounding Co-authored-by: Wilco Fiers <[email protected]>
|
5f25497
to
9430193
Compare
9430193
to
8ea779b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Release Audit
- fix(nativeSelectValue): update selected value on change #3154
- chore(p-as-heading): add
margin
description back tocheck-options.md
#3180 Should have been docs, not chore. Minor issue - fix(aria-prohibited-attr): update metadata message #3206
Note: We added the "tests: fix for release" commit to the release outside a PR in order to fix failing tests on this release. The tests fail because some of the changes were not picked into this release.
8ea779b
to
e1cd670
Compare
One more commit: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SRI update reviewed for security too.
Cherry picking some
fixes
had code from a fewfeat
prs, so had to manually edit the files to remove the new changes and tests.