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

chore(release): 4.3.4 #3218

Merged
merged 31 commits into from
Oct 25, 2021
Merged

chore(release): 4.3.4 #3218

merged 31 commits into from
Oct 25, 2021

Conversation

straker
Copy link
Contributor

@straker straker commented Oct 18, 2021

Cherry picking some fixes had code from a few feat prs, so had to manually edit the files to remove the new changes and tests.

straker and others added 27 commits October 18, 2021 09:50
…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]>
@straker straker requested a review from a team as a code owner October 18, 2021 15:55
@CLAassistant
Copy link

CLAassistant commented Oct 18, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
7 out of 9 committers have signed the CLA.

✅ straker
✅ forsti0506
✅ dylanb
✅ Zidious
✅ scottaohara
✅ macjohnny
✅ WilcoFiers
❌ kevin940726
❌ kaelig
You have signed the CLA already but the status is still pending? Let us recheck it.

WilcoFiers
WilcoFiers previously approved these changes Oct 19, 2021
Copy link
Contributor

@WilcoFiers WilcoFiers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Release Audit

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.

@WilcoFiers
Copy link
Contributor

WilcoFiers
WilcoFiers previously approved these changes Oct 25, 2021
Copy link
Contributor

@WilcoFiers WilcoFiers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wr

Copy link
Contributor

@WilcoFiers WilcoFiers left a 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.

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.

10 participants