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

feat: accessibility improvements #1694

Open
wants to merge 29 commits into
base: develop
Choose a base branch
from

Conversation

LucasHengelhaupt
Copy link
Contributor

@LucasHengelhaupt LucasHengelhaupt commented Jul 4, 2024

This PR adds accessibility improvements.

PR Type

[x] Feature

What Is the Current Behavior?

Currently, the Intershop Progressive Web App (PWA) has accessibility issues related to color contrast, link identification, tab-focus visibility, semantic order of HTML elements, missing ARIA attributes, touch target sizes and others.

What Is the New Behavior?

The following accessibility issues are fixed:

Does this PR Introduce a Breaking Change?

[x] Yes

Open Issues

[ ] check changes of the wishlist feature: #1590 (esp. animated buttons on PDP and "account.wishlists.shared_wishlist.error")
[ ] check b2c favicon

Other Information

AB#97976

@LucasHengelhaupt LucasHengelhaupt added the feature New feature or request label Jul 4, 2024
@LucasHengelhaupt LucasHengelhaupt requested a review from SGrueber July 4, 2024 10:07
@LucasHengelhaupt LucasHengelhaupt self-assigned this Jul 4, 2024
@LucasHengelhaupt LucasHengelhaupt force-pushed the feat/accessibility_improvements branch from 43b0ea5 to 41cc281 Compare July 17, 2024 13:24
@LucasHengelhaupt LucasHengelhaupt force-pushed the feat/accessibility_improvements branch 4 times, most recently from 0dfaac4 to 49232f8 Compare August 6, 2024 10:25
@shauke shauke added this to the 6.0 milestone Aug 14, 2024
@andreassteinmann andreassteinmann force-pushed the feat/accessibility_improvements branch from 41023e4 to 913cb20 Compare August 16, 2024 13:24
Copy link

Azure Demo Servers are available:

11 similar comments
Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

@shauke shauke marked this pull request as draft August 22, 2024 07:09
Copy link

Azure Demo Servers are available:

3 similar comments
Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

Copy link

Azure Demo Servers are available:

@andreassteinmann andreassteinmann force-pushed the feat/accessibility_improvements branch from e266a26 to 54f05b6 Compare August 26, 2024 07:53
@LucasHengelhaupt LucasHengelhaupt force-pushed the feat/accessibility_improvements branch from 31f038b to 7fd6064 Compare August 27, 2024 14:27
SGrueber added a commit that referenced this pull request Feb 6, 2025
…relying on color alone, update underline behavior (#1694)
SGrueber added a commit that referenced this pull request Feb 7, 2025
@LucasHengelhaupt LucasHengelhaupt force-pushed the feat/accessibility_improvements branch from 96e05b2 to b0cef6e Compare February 10, 2025 16:25
@LucasHengelhaupt LucasHengelhaupt force-pushed the feat/accessibility_improvements branch from caa690b to 2bc3b36 Compare February 10, 2025 16:46
Copy link

Azure Demo Servers are available:

LucasHengelhaupt added a commit that referenced this pull request Feb 11, 2025
@LucasHengelhaupt LucasHengelhaupt force-pushed the feat/accessibility_improvements branch from 07127e6 to 4944351 Compare February 11, 2025 12:39
SGrueber pushed a commit that referenced this pull request Feb 14, 2025
…ssword component and update error indicators (#1694)

* Add functionality to toggle password visibility for password fields.
* Replace TextInputFieldComponent with PasswordFieldComponent to provide password-specific functionality.
* Update error field indicator from "x" to "!" for improved clarity and accessibility.

BREAKING CHANGES: The TextInputFieldComponent is no longer used for password fields. Instead, the PasswordFieldComponent is now used, which includes functionality to toggle password visibility. Password validators are applied to each instance as needed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request has conflicts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants