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

[4.0.x] Update skipto #35590

Merged
merged 1 commit into from
Sep 18, 2021
Merged

[4.0.x] Update skipto #35590

merged 1 commit into from
Sep 18, 2021

Conversation

brianteeman
Copy link
Contributor

This updates the skipto js to the newly released 4.1.2 - nothing spectacular changed apart from fixing an issue with WAVE tester making a false error report

Version 4.1.2

Added aria-busy="true" attribute to menu element when SkipTo is initialized and being updated with new menu items to support validators looking for required menu items for the menu role.
Added the optional aria-controls attribute to button element to reference the id of the menu element as defined in the W3C ARIA Authoring practices for menu button pattern.

Version 4.1.1

Removed aria-describedby from button, since screen readers read the accesskey information.

This updates the skipto js to the newly released 4.1.2

### Version 4.1.2
Added aria-busy="true" attribute to menu element when SkipTo is initialized and being updated with new menu items to support validators looking for required menu items for the menu role.
Added the optional aria-controls attribute to button element to reference the id of the menu element as defined in the W3C ARIA Authoring practices for menu button pattern.

### Version 4.1.1
Removed aria-describedby from button, since screen readers read the accesskey information.
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Sep 17, 2021
@wilsonge wilsonge merged commit 0103d14 into joomla:4.0-dev Sep 18, 2021
@wilsonge
Copy link
Contributor

Thanks!

@wilsonge wilsonge added this to the Joomla 4.0.4 milestone Sep 18, 2021
@brianteeman
Copy link
Contributor Author

thanks

@brianteeman brianteeman deleted the skipto412 branch September 18, 2021 19:47
brianteeman added a commit to brianteeman/joomla-cms that referenced this pull request Oct 3, 2021
This updates the skipto js to the newly released 4.1.2

### Version 4.1.2
Added aria-busy="true" attribute to menu element when SkipTo is initialized and being updated with new menu items to support validators looking for required menu items for the menu role.
Added the optional aria-controls attribute to button element to reference the id of the menu element as defined in the W3C ARIA Authoring practices for menu button pattern.

### Version 4.1.1
Removed aria-describedby from button, since screen readers read the accesskey information.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NPM Resource Changed This Pull Request can't be tested by Patchtester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants