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

Refactor New Widget & Quick Post options menu to use disclosure pattern #11611

Merged
merged 8 commits into from
Aug 12, 2022

Conversation

gcamacho079
Copy link
Contributor

@gcamacho079 gcamacho079 commented Jul 15, 2022

Description

  • Refactor new widget dropdown to use disclosure menu (Resolves DEV-746, DEV-951)
  • Refactor quick post options menu to use disclosure menu (Resolves DEV-275, DEV-276)

@gcamacho079 gcamacho079 added the accessibility 👤 features related to accessibility label Jul 15, 2022
@brandonkelly brandonkelly changed the base branch from 4.2 to 4.3 July 27, 2022 22:48
@linear
Copy link

linear bot commented Aug 1, 2022

DEV-951 New widget button on Dashboard doesn't have accessible name

Page Area:

Dashboard - #newwidgetmenubtn

Issue Description:

As of 4.1.0, no accessible name is present on the button to add a new widget

Action:

Refactor to disclosure widget and/or add aria-label/aria-labelledby

Screenshot/Code Snippet:

Screen Shot 2022-07-28 at 1.30.14 PM.png

Resolves CMS-251

# Conflicts:
#	src/web/assets/admintable/dist/css/app.css
#	src/web/assets/admintable/dist/js/app.js
#	src/web/assets/admintable/dist/js/app.js.map
#	src/web/assets/dashboard/dist/Dashboard.js
#	src/web/assets/dashboard/dist/Dashboard.js.map
#	src/web/assets/dashboard/src/Dashboard.js
#	src/web/assets/pluginstore/dist/css/app.css
#	src/web/assets/pluginstore/dist/css/app.css.map
#	src/web/assets/quickpost/dist/QuickPostWidget.js
#	src/web/assets/quickpost/dist/QuickPostWidget.js.map
[ci skip]
# Conflicts:
#	CHANGELOG-WIP.md

[ci skip]
@brandonkelly brandonkelly merged commit 6f3b5b2 into 4.3 Aug 12, 2022
@brandonkelly brandonkelly deleted the a11y/disclosures branch August 12, 2022 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility 👤 features related to accessibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants