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: Update styles for dark backgrounds to be readable #6066

Merged
merged 8 commits into from
Jan 6, 2025

Conversation

jennifer-shehane
Copy link
Member

@jennifer-shehane jennifer-shehane commented Jan 3, 2025

Some of the new changes were not displaying well in dark mode. I've updated them to be more readable.

Additionally, this fixes the tests so that they truly visit all URLs and that all URLs are visited in dark mode (so we could see any decrease in Accessibility scores for light and dark mode).

Accessibility Report: https://cloud.cypress.io/projects/imown1/runs/230/accessibility

Before

Screenshot 2025-01-03 at 12 58 45 PM
Screenshot 2025-01-03 at 12 49 55 PM

After

Screenshot 2025-01-03 at 12 58 37 PM
Screenshot 2025-01-03 at 1 00 35 PM

Copy link

cypress bot commented Jan 3, 2025

cypress-documentation    Run #236

Run Properties:  status check passed Passed #236  •  git commit f9c21dd78e: Add test for 404 display
Project cypress-documentation
Branch Review card-dark-bg-fix
Run status status check passed Passed #236
Run duration 06m 12s
Commit git commit f9c21dd78e: Add test for 404 display
Committer Jennifer Shehane
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 290
View all changes introduced in this branch ↗︎
UI Coverage  11.12%
  Untested elements 2265  
  Tested elements 3  
Accessibility  95.89%
  Failed rules  2 critical   5 serious   4 moderate   0 minor
  Failed elements 687  

@jennifer-shehane jennifer-shehane self-assigned this Jan 3, 2025
@jennifer-shehane jennifer-shehane merged commit 3fd06e1 into main Jan 6, 2025
10 checks passed
@jennifer-shehane jennifer-shehane deleted the card-dark-bg-fix branch January 6, 2025 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant