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

Accessibility: Grid filter drop down can't be accessed via keyboard #5963

Merged
merged 12 commits into from
Sep 5, 2019
Merged

Accessibility: Grid filter drop down can't be accessed via keyboard #5963

merged 12 commits into from
Sep 5, 2019

Conversation

BatJan
Copy link
Contributor

@BatJan BatJan commented Jul 20, 2019

Prerequisites

  • I have added steps to test this contribution in the description below

This fixes issues 62,82 and 87 from #5277

Description

This PR makes the grid selections keyboard accessible and it also improves accessibility for screen readers as well exposing visually hidden texts.

Before it was not possible to tab to the layout dropdown - Now it's possible to reach the layout dropdown toggle using the keyboard and tab through the options as well. When one tabs outside the options that appear the dropdown is closed due to the addition of a deep-blur directive, which is also mentioned in #5960 - It's cherry picked from this branch.

I have also expanded the language files with a "visuallyHiddenTexts" area where I hope we can agree to store text key's that serve pure non-visual purposes, which I think will make it easier to ensure translations into other languages etc. 😃

Looking forward to receive some feedback 👍

Content section before
content-grid-dropdown-before

Content section after
content-grid-dropdown-after

Media section before
media-grid-dropdown-before

Media section after
media-grid-dropdown-after

Recycle bin before
This is displaying the recycle bin from "Content" section but it's the same for the recycle bin in the "Media" section.

recycle-bin-grid-dropdown-before

Recycle bin after
recycle-bin-grid-dropdown-after

@emmaburstow
Copy link
Contributor

Lovely work again, Jan. Thanks for putting the time aside. Keyboard accessibility is going to make such a difference.

Em

# Conflicts:
#	src/Umbraco.Web.UI/Umbraco/config/lang/da.xml
#	src/Umbraco.Web.UI/Umbraco/config/lang/en.xml
#	src/Umbraco.Web.UI/Umbraco/config/lang/en_us.xml
@nul800sebastiaan
Copy link
Member

Lovely work @BatJan ! Remember to remove those console logs next time! 😉

@nul800sebastiaan nul800sebastiaan merged commit 4b344ac into umbraco:v8/dev Sep 5, 2019
nul800sebastiaan pushed a commit that referenced this pull request Sep 5, 2019
…5963)

(cherry picked from commit 4b344ac)

# Conflicts:
#	src/Umbraco.Web.UI/Umbraco/config/lang/da.xml
#	src/Umbraco.Web.UI/Umbraco/config/lang/en.xml
#	src/Umbraco.Web.UI/Umbraco/config/lang/en_us.xml
nul800sebastiaan added a commit that referenced this pull request Sep 5, 2019
…yboard (#5963)"

This reverts commit e1a438d.

Should not have been picked into a patch release :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants