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: Media uploader widget can't be accessed via keyboard #5944

Merged
merged 6 commits into from
Aug 14, 2019
Merged

Accessibility: Media uploader widget can't be accessed via keyboard #5944

merged 6 commits into from
Aug 14, 2019

Conversation

BatJan
Copy link
Contributor

@BatJan BatJan commented Jul 18, 2019

Prerequisites

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

This fixes issue 88 from #5277

Description

In this PR I have changed the

to a so it can be reached by keyboard and fixed the styling accordingly.

Apart from that I have added a text above the image to provide some more context about what is expected of the user here - I felt that the "- or click here..." was not providing enough context. Especially for editors as well.

However I'm open for suggestions for the text I have added since I'm not sure if my current suggestion is good enough. But I think it's better than having nothing. I'm not sure adding a text like "Drag and drop your files onto the image/dropzone below" is properly accessible - But neither is my current suggestion. But as mentioned, I think it's better than nothing 😃

Below we have the before and after GIF's.

Before
media-widget-before

After
media-widget-after

@BatJan BatJan marked this pull request as ready for review July 18, 2019 19:34
@BatJan BatJan changed the title Accessibility: Issue 88 - Media uploader widget can't be accessed via keyboard Accessibility: Media uploader widget can't be accessed via keyboard Jul 21, 2019
@emmaburstow
Copy link
Contributor

Lovely work, Jan. Another win for accessibility. We'll shout if we need you,

Em

@BatJan
Copy link
Contributor Author

BatJan commented Jul 23, 2019

Thank you @emmaburstow 😃 However I think I ended up changing the added paragraph to say "area" instead of "Canvas" - I'm open for textual suggestions here 😄

# 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

Awesome, thanks @BatJan ! 👍

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