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
FEATURE: filter admin sidebar #25853
Conversation
Ability to filter admin sidebar. Filter can be cleared. In addition, it can be accessed with ctrl+/ shortcut
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.
Working nicely, just needs a few tweaks. Very happy with the speed of filtering! Visual bug, the X goes off to the side:
I think also when you click the X we should just re-focus the search box.
On mobile, we need to align the search better:
Maybe we will not even show this on mobile...not sure yet but at the least it needs alignment.
app/assets/javascripts/discourse/app/components/sidebar/filter.gjs
Outdated
Show resolved
Hide resolved
app/assets/javascripts/discourse/app/components/sidebar/filter.gjs
Outdated
Show resolved
Hide resolved
app/assets/javascripts/discourse/app/components/sidebar/filter.gjs
Outdated
Show resolved
Hide resolved
{{#if this.shouldDisplay}} |
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.
It's super fast ATM doing this, so it's not necessary, but have you tried adding a css class that does display:none instead of adding/removing elements as David suggested? Just curious.
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.
No, once it became fast enough, I stopped improving
app/assets/javascripts/discourse/app/components/sidebar/api-section.js
Outdated
Show resolved
Hide resolved
app/assets/javascripts/discourse/app/components/sidebar/api-section.js
Outdated
Show resolved
Hide resolved
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.
app/assets/javascripts/discourse/app/components/sidebar/api-section.js
Outdated
Show resolved
Hide resolved
app/assets/javascripts/discourse/app/components/sidebar/filter-no-results.gjs
Show resolved
Hide resolved
app/assets/javascripts/discourse/app/components/sidebar/filter.gjs
Outdated
Show resolved
Hide resolved
…ction.js Co-authored-by: Martin Brennan <mjrbrennan@gmail.com>
Co-authored-by: Martin Brennan <mjrbrennan@gmail.com>
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.
Amazing work 👏
Ability to filter admin sidebar.
The filter can be cleared. In addition, it can be accessed with ctrl+/ shortcut
Desktop demo
https://github.com/discourse/discourse/assets/72780/e3f2d22f-fff0-4770-aefc-00348d539672
Mobile: