nodejs.org icon indicating copy to clipboard operation
nodejs.org copied to clipboard

feat(tabs): fix contrast ratio

Open AugustinMauroy opened this issue 1 year ago • 5 comments

Description

To solve issue I had follow Figma design. And allow forcing dark theme to allow codeTabs to force theme.

Validation

Related Issues

Fixe #6527

Check List

  • [X] I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • [X] I have run npx turbo format to ensure the code follows the style guide.
  • [X] I have run npx turbo test to check if all tests are passing.
  • [X] I have run npx turbo build to check if the website builds without errors.
  • NA I've covered new added functionality with unit tests if necessary.

AugustinMauroy avatar Mar 25 '24 09:03 AugustinMauroy

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview Mar 28, 2024 9:17am

vercel[bot] avatar Mar 25 '24 09:03 vercel[bot]

Lighthouse Results

URL Performance Accessibility Best Practices SEO Report
/en 🟢 99 🟢 90 🟢 100 🟢 91 🔗
/en/about 🟢 98 🟢 91 🟢 100 🟢 91 🔗
/en/about/previous-releases 🟢 99 🟢 90 🟢 100 🟢 92 🔗
/en/download 🟢 99 🟠 89 🟢 96 🟢 91 🔗
/en/blog 🟢 100 🟢 90 🟢 100 🟢 92 🔗

github-actions[bot] avatar Mar 25 '24 09:03 github-actions[bot]

Unit Test Coverage Report

Lines Statements Branches Functions
Coverage: 84%
80.1% (451/563) 79.44% (143/180) 71.17% (79/111)

Unit Test Report

Tests Skipped Failures Errors Time
90 0 :zzz: 0 :x: 0 :fire: 4.436s :stopwatch:

github-actions[bot] avatar Mar 25 '24 09:03 github-actions[bot]

Screenshots of before/after pls?

ovflowd avatar Mar 28 '24 11:03 ovflowd

Screenshots of before/after pls?

yep _note there are no change on dark mode and on tabs comment _

before

Capture d’écran 2024-03-28 à 12 43 51

after

Capture d’écran 2024-03-28 à 12 45 08

AugustinMauroy avatar Mar 28 '24 11:03 AugustinMauroy

@ovflowd 👋 can we merge ?

AugustinMauroy avatar Apr 02 '24 10:04 AugustinMauroy

Lol, sure, go ahead ;)

ovflowd avatar Apr 02 '24 10:04 ovflowd