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

{Freeway-Bug}[PWA Builder]: Color contrast ratio fails for the URL after pasting inside the 'enter the URL' edit field. #4743

Closed
Anshi0420 opened this issue Jun 20, 2024 · 15 comments
Assignees
Labels
A11ySev2 Accessibility Team tag - P2 items A11yWCAG Accessibility Team tag - Web Content Accessibility Guidelines Accessibility [E+D] Issues found by the accessibility E+D Team at Microsoft bug 🐛 cannot reproduce CT-June24 HCL-E+D needs attention 👋 needs triage 🔍 Product-PWABuilder For Tracking PWABuilder bugs by Accessibility E+D team PWABuilder_Web

Comments

@Anshi0420
Copy link
Collaborator

What happened?

When a URL is pasted into the 'enter the URL' edit field, the color contrast ratio between the text color and the background fails to meet accessibility standards i.e., 2.8:1.

How do we reproduce the behavior?

Repro Steps:

  1. Open the above URL.
  2. Navigate to the 'Enter URL' edit field.
  3. Pase any URL.
  4. Check the color contrast ratio with the color contrast analyzer tool.
  5. Observe the issue.

What do you expect to happen?

The text color and background should ensure a sufficient color contrast ratio (typically 4.5:1 for normal text) after a URL is pasted into the edit field.

What environment were you using?

Test Environment:
OS Build: Windows 11 Enterprise Insider Preview 24H2 (26120.461)
Edge Browser: Version 123.0.2400.1 (Official build) dev (64-bit)
URL: https://www.pwabuilder.com/
Tool: Color contrast Analyzer

Additional context

User Impact:
Users with visual impairments, such as low vision or color blindness, may find it difficult or impossible to read the pasted URL in the edit field due to inadequate color contrast.

WCAG Reference:
https://www.w3.org/WAI/WCAG22/Understanding/contrast-minimum

Attachments:
MAS1 4 3_Color contrast ratio fails for the URL after pasting inside the 'enter the url' edit field

@Anshi0420 Anshi0420 added bug 🐛 needs triage 🔍 A11yWCAG Accessibility Team tag - Web Content Accessibility Guidelines Accessibility [E+D] Issues found by the accessibility E+D Team at Microsoft A11ySev2 Accessibility Team tag - P2 items HCL-E+D Product-PWABuilder For Tracking PWABuilder bugs by Accessibility E+D team labels Jun 20, 2024
@Jaylyn-Barbee Jaylyn-Barbee changed the title [PWA Builder-Blog]: Color contrast ratio fails for the URL after pasting inside the 'enter the URL' edit field. [PWA Builder]: Color contrast ratio fails for the URL after pasting inside the 'enter the URL' edit field. Jun 20, 2024
@Anshi0420 Anshi0420 changed the title [PWA Builder]: Color contrast ratio fails for the URL after pasting inside the 'enter the URL' edit field. {Freeway-Bug}[PWA Builder]: Color contrast ratio fails for the URL after pasting inside the 'enter the URL' edit field. Jun 27, 2024
Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

4 similar comments
Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

jgw96 added a commit that referenced this issue Aug 27, 2024
@jgw96 jgw96 mentioned this issue Aug 27, 2024
3 tasks
jgw96 added a commit that referenced this issue Aug 27, 2024
fixes #4743 
<!-- Link to relevant issue (for ex: "fixes #1234") which will
automatically close the issue once the PR is merged -->

## PR Type
<!-- Please uncomment one ore more that apply to this PR -->

Bugfix
<!-- - Feature -->
<!-- - Code style update (formatting) -->
<!-- - Refactoring (no functional changes, no api changes) -->
<!-- - Build or CI related changes -->
<!-- - Documentation content changes -->
<!-- - Sample app changes -->
<!-- - Other... Please describe: -->


## Describe the current behavior?
<!-- Please describe the current behavior that is being modified or link
to a relevant issue. -->
When the input was autofilled, the color was a bright blue that did not
have good contrast.

## Describe the new behavior?
It now uses our standard input color.

## PR Checklist

- [ ] Test: run `npm run test` and ensure that all tests pass
- [ ] Target main branch (or an appropriate release branch if
appropriate for a bug fix)
- [ ] Ensure that your contribution follows [standard accessibility
guidelines](https://docs.microsoft.com/en-us/microsoft-edge/accessibility/design).
Use tools like https://webhint.io/ to validate your changes.


## Additional Information
Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

2 similar comments
Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

4 similar comments
Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

Copy link
Contributor

This issue has been marked as "needs attention 👋" since it has not been triaged for 7 days. Please triage the issue .

@Jaylyn-Barbee Jaylyn-Barbee moved this from Blocked 🔒 to Todo 📃 in PWABuilder Oct 28, 2024
@Jaylyn-Barbee
Copy link
Contributor

Fixed by #4797

@github-project-automation github-project-automation bot moved this from Todo 📃 to Done ✔️ in PWABuilder Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A11ySev2 Accessibility Team tag - P2 items A11yWCAG Accessibility Team tag - Web Content Accessibility Guidelines Accessibility [E+D] Issues found by the accessibility E+D Team at Microsoft bug 🐛 cannot reproduce CT-June24 HCL-E+D needs attention 👋 needs triage 🔍 Product-PWABuilder For Tracking PWABuilder bugs by Accessibility E+D team PWABuilder_Web
Projects
Status: Done ✔️
Development

No branches or pull requests

4 participants