Skip to content

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