Skip to content

Conversation

reakaleek
Copy link
Member

@reakaleek reakaleek commented Feb 21, 2025

Closes #580

Context

The sub lists of the navigation items had height: 0 and were expanded based on the height. This was done because the initial idea was to expand with an animation.

Change

Instead of hiding with height: 0 use display: hidden, this way it's excluded from the browser search.

@reakaleek reakaleek self-assigned this Feb 21, 2025
@reakaleek reakaleek requested a review from a team February 21, 2025 21:24
@reakaleek reakaleek added the fix label Feb 21, 2025
Copy link
Member

@bmorelli25 bmorelli25 left a comment

Choose a reason for hiding this comment

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

tested and works for me!

@reakaleek reakaleek merged commit ab1ca67 into main Feb 21, 2025
5 of 6 checks passed
@reakaleek reakaleek deleted the feature/exclude-nav-items-search branch February 21, 2025 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ctrl+F searches collapsed nav items with no user feedback
2 participants