Closed yashsaraswat2004 closed 1 month ago
The changes made in this pull request involve a modification to the hover state CSS class for menu block links in the MenuBlocks
component. The hover class has been updated from hover:bg-gray-50
to hover:bg-gray-300
, enhancing the visual feedback for users when they hover over menu items. No other logic or functionality within the component has been altered.
File | Change Summary |
---|---|
components/navigation/MenuBlocks.tsx | Updated hover class from hover:bg-gray-50 to hover:bg-gray-300 for menu block links. |
Objective | Addressed | Explanation |
---|---|---|
Background Color should be visible on hovering in the dropdown menu (3330) | ✅ |
🐇 In the menu, the colors now play,
Hovering brings a brighter day!
From gray-50 to gray-300, so fine,
A change in hue, oh how it shines!
With each click and every glance,
The dropdown dances, a joyful chance! 🌈
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Built without sensitive environment variables
Name | Link |
---|---|
Latest commit | eca8f6019a52c8e86c60e3853ee673b3e36c366d |
Latest deploy log | https://app.netlify.com/sites/asyncapi-website/deploys/671fb332f01465000826ce1c |
Deploy Preview | https://deploy-preview-3340--asyncapi-website.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
⚡️ Lighthouse report for the changes in this PR: | Category | Score |
---|---|---|
🔴 Performance | 45 | |
🟢 Accessibility | 98 | |
🟢 Best practices | 92 | |
🟢 SEO | 100 | |
🔴 PWA | 33 |
Lighthouse ran on https://deploy-preview-3340--asyncapi-website.netlify.app/
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 49.14%. Comparing base (
d7ee919
) to head (eca8f60
). Report is 2 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The previous color seems relevant too. Why we need more intense color on hover?
This PR fixes an issue with the visibility of background colors on components in the dropdown menu when hovering. Previously, hovering over dropdown items within the Docs, Community, and Tools menus did not display the intended background color, causing a poor user experience.
To address this:
Updated the background color styling to improve contrast and visibility for dropdown items on hover. Verified that changes are consistent across different dropdown menus for a cohesive look. Screenshots
How to Test
Open the ASyncAPI website in Google Chrome. Hover over Docs, Community, or Tools in the navigation bar to trigger the dropdown menu. Move the cursor over items in the dropdown menu to see the hover effect. Related Issue(s)
Fixes #3330
Summary by CodeRabbit