Skip to content

Conversation

@tagliala
Copy link
Contributor

@tagliala tagliala commented Nov 5, 2024

Address accessibility issues identified by Lighthouse, focusing on
improving the readability of form hint text in dark mode.

  • Enhance readability by making extra-small text darker in light mode
  • Refine dark theme color palette for improved contrast on non-black
    backgrounds:
    • Change some text colors from gray-500 to gray-400
    • Update link color from blue-600 to blue-500

Before vs After

Form (dark) (Lighthouse Score: 92 -> 96)

URL: http://localhost:5000/admin/stores/new

- f.input :name
+ f.input :name, hint: 'Enter name'

image image

Lighthouse Report

Low-contrast text is difficult or impossible for many users to read. Learn how to provide sufficient color contrast.

Failing Elements

  • p.inline-hints

Comment (light) (Lighthouse Score: 88 -> 88)

URL: http://localhost:5000/admin/categories/2

image image

Comment (dark) (Lighthouse Score: 88 -> 88)

URL: http://localhost:5000/admin/categories/2

image image

Lighthouse Report

Low-contrast text is difficult or impossible for many users to read. Learn how to provide sufficient color contrast.

Failing Elements

  • a

@codecov
Copy link

codecov bot commented Nov 5, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.08%. Comparing base (95a07fa) to head (18222b7).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #8535   +/-   ##
=======================================
  Coverage   99.08%   99.08%           
=======================================
  Files         139      139           
  Lines        4043     4043           
=======================================
  Hits         4006     4006           
  Misses         37       37           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tagliala tagliala force-pushed the chore/improve-text-contrast-ratio branch from 5e126e2 to 7514b11 Compare January 8, 2025 22:13
@tagliala tagliala force-pushed the chore/improve-text-contrast-ratio branch 2 times, most recently from 3c221ca to c350e61 Compare June 23, 2025 14:12
@tagliala tagliala requested a review from javierjulio June 25, 2025 12:12
@tagliala tagliala force-pushed the chore/improve-text-contrast-ratio branch from c350e61 to 2dc9983 Compare July 6, 2025 09:42
@tagliala tagliala force-pushed the chore/improve-text-contrast-ratio branch 2 times, most recently from 8ae1397 to 9498eb2 Compare July 23, 2025 08:03
@javierjulio javierjulio force-pushed the chore/improve-text-contrast-ratio branch 2 times, most recently from af5cc45 to 3db9c13 Compare September 9, 2025 19:18
Address accessibility issues identified by Lighthouse, focusing on
improving the readability of form hint text in dark mode.

- Enhance readability by making extra-small text darker in light mode
- Refine dark theme color palette for improved contrast on non-black
  backgrounds:  
  - Change some text colors from `gray-500` to `gray-400`
  - Update link color from `blue-600` to `blue-500`
@javierjulio javierjulio force-pushed the chore/improve-text-contrast-ratio branch from 3db9c13 to 18222b7 Compare September 9, 2025 19:27
@javierjulio javierjulio changed the title Improve accessibility with better color contrast Improve text color contrast Sep 9, 2025
Copy link
Member

@javierjulio javierjulio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@javierjulio javierjulio merged commit 1debdfc into master Sep 9, 2025
25 checks passed
@javierjulio javierjulio deleted the chore/improve-text-contrast-ratio branch September 9, 2025 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants