-
Notifications
You must be signed in to change notification settings - Fork 157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add search nav item #7210
Add search nav item #7210
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love the simplification, and works nicely. Let's get this through CI and squash-merge 🕵🏽
SonarCloud Quality Gate failed. |
I don't get the userstory for this feature (new Item "Search" appears). I'd expect that there is no active sidebar item at all - first proposal of @kulmann s comment: #7190 (comment) |
Well this was spoken with @kulmann which we found a good idea, easy to remove anyways if you wish so. |
If I'm not missing something we should not have an additional Item ( remove it) and follow Benedikts first proposal -> have no item at all for search in the left sidebar as I guess this is the users expectation (looking at eg. webmail clients or competitors). We can wait until Benedikt is back - maybe I miss something... |
Let's remove it, but the issue is not solved then, nav highlighter is still behaving strange |
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist:
Open tasks: