Skip to content

Conversation

@this-is-varunxyz
Copy link
Contributor

@this-is-varunxyz this-is-varunxyz commented Oct 16, 2025

This PR implements a hover-to-open feature for the main navigation bar on desktop-sized screens to improve usability. #1766

Key improvements:

  • Menus now open on hover with a smooth slide-down animation.
  • On desktop, the default click action on the main links is disabled to prevent confusion and remove the :focus state.
  • The feature is disabled on mobile viewports (<768px), preserving the default tap-to-open behavior.

@netlify
Copy link

netlify bot commented Oct 16, 2025

Deploy Preview for earnest-hotteok-b1e1bf ready!

Name Link
🔨 Latest commit 2b76e7b
🔍 Latest deploy log https://app.netlify.com/projects/earnest-hotteok-b1e1bf/deploys/68f0d9ce71ae5c0008ea77f5
😎 Deploy Preview https://deploy-preview-1768--earnest-hotteok-b1e1bf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@eduardo-rodrigues
Copy link
Member

Hi @this-is-varunxyz. This is looking very good. I checked the effect on a couple of browsers and my phone, and all goes as expected. I requested that a couple of other colleagues from the team look at this PR since it changes the feeling of the site a bit.

N.B.: if you took the code from somewhere, do add a comment with the source, to be fair.

Thank you for this 👍!

@this-is-varunxyz
Copy link
Contributor Author

this-is-varunxyz commented Oct 16, 2025

thank you sir , @eduardo-rodrigues
Yes, the code is original. I did used Gemini to generate the comments to ensure that the logic is easy for others to understand.

Copy link
Member

@graeme-a-stewart graeme-a-stewart left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @this-is-varunxyz

@eduardo-rodrigues eduardo-rodrigues merged commit 9b0e1dd into HSF:main Oct 17, 2025
8 checks passed
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