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

fix: Changelog trigger and minor visual bugs

Open canerakdas opened this issue 10 months ago • 3 comments

Description

This PR aims to solve some minor visual problems on mobile and the SEO problem in the change log trigger.

  • The anchor element in the changelog trigger has been removed, and @radix-ui/react-slot has been added, which we can use in such cases.
  • Fixed the overflow issue in the changelog modal
  • Underline styling in headings in mobile resolution has been removed
  • The semibold font arrangement was made instead of the underlined links we added for accessibility in mobile design. IMO we do not lose anything in accessibility and the content looks more focused.

Validation

Badge Component

image image

Headings

image image

Changelog modal

image image

In Preview we need to make sure everything is working correctly on mobile/desktop and our SEO scores are improving

Related Issues

Fixes #6566

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.
  • [x] I've covered new added functionality with unit tests if necessary.

canerakdas avatar Mar 27 '24 08:03 canerakdas