Skip to content

Add missing background to collapsible menu items#4940

Merged
Simek merged 3 commits into
react:mainfrom
clicktodev:hover-effect-menu-item
Jan 1, 2026
Merged

Add missing background to collapsible menu items#4940
Simek merged 3 commits into
react:mainfrom
clicktodev:hover-effect-menu-item

Conversation

@clicktodev

@clicktodev clicktodev commented Jan 1, 2026

Copy link
Copy Markdown
Contributor

Add a hover bg style to the collapsible menu items.

image

@meta-cla meta-cla Bot added the CLA Signed label Jan 1, 2026
@netlify

netlify Bot commented Jan 1, 2026

Copy link
Copy Markdown

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit dc516ed
🔍 Latest deploy log https://app.netlify.com/projects/react-native/deploys/6956b487de96a700080cf994
😎 Deploy Preview https://deploy-preview-4940--react-native.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.

@Simek

Simek commented Jan 1, 2026

Copy link
Copy Markdown
Collaborator

Hey @clicktodev, this seems like a nice improvement, however after the changes the non-collapsible menu items are also highlighted on hover, which is not correct.

Screenshot 2026-01-01 155708

@clicktodev

Copy link
Copy Markdown
Contributor Author

fixed

@Simek Simek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the follow up fix, LGTM! 🚀

@Simek Simek merged commit e94adcb into react:main Jan 1, 2026
4 checks passed
@clicktodev clicktodev deleted the hover-effect-menu-item branch January 2, 2026 21:30
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.

2 participants