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

Fix instance of color-only error indication on elements #11610

Merged
merged 3 commits into from
Aug 12, 2022

Conversation

gcamacho079
Copy link
Contributor

Description

  • Use alert icon next to element title when it has validation errors

Screen Shot 2022-07-15 at 10 29 56 AM

Resolves DEV-687

@gcamacho079 gcamacho079 added the accessibility 👤 features related to accessibility label Jul 15, 2022
@linear
Copy link

linear bot commented Jul 15, 2022

DEV-687 When an image asset has a validation error, it's indicated via red text inside the entry form

Page Area:

Entry form

Issue Description:

When a field inside a selected element has a validation error, the validation error is only indicated on the element via the red text color.

Action:

Add an error symbol or other indicator that doesn't rely on the use of color alone

Screenshot/Code Snippet:

Screenshot showing the selected asset (inside the image matrix block, not the slideout) only indicates there's an error using red text.

Screen Shot 2022-06-02 at 2.10.16 PM.png

Resolves CMS-234

@brandonkelly brandonkelly changed the base branch from 4.2 to 4.3 July 27, 2022 22:48
# Conflicts:
#	src/web/assets/admintable/dist/css/app.css
#	src/web/assets/admintable/dist/css/app.css.map
#	src/web/assets/admintable/dist/js/app.js
#	src/web/assets/admintable/dist/js/app.js.map
#	src/web/assets/cp/dist/css/cp.css
#	src/web/assets/cp/dist/css/cp.css.map
#	src/web/assets/pluginstore/dist/css/app.css.map
[ci skip]
@brandonkelly brandonkelly merged commit 689395f into 4.3 Aug 12, 2022
@brandonkelly brandonkelly deleted the a11y/fix-color-only-error branch August 12, 2022 16:55
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