-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Enhancement: Sidebar open/close classes #3855
Conversation
… open/closed state
Why move custom classes to middle or end of classes list? |
Weird. those shouldn't have changed. Let me see what happened. Good catch. |
Looks like there was an extension overriding the sort order. Latest push corrects the order. |
Thanks for noticing this! (Always nice to have an extra pair of eyes making sure the order stays correct in the codebase. 👀) |
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.
Filament CSS classes should be first in the class list and be prefixed with filament-
. 🙂
packages/admin/resources/views/components/layouts/app.blade.php
Outdated
Show resolved
Hide resolved
packages/admin/resources/views/components/layouts/app/sidebar/index.blade.php
Outdated
Show resolved
Hide resolved
packages/admin/resources/views/components/layouts/app/sidebar/index.blade.php
Outdated
Show resolved
Hide resolved
Co-authored-by: Zep Fietje <hey@zepfietje.com>
…index.blade.php Co-authored-by: Zep Fietje <hey@zepfietje.com>
Adds classes to sidebar and filament main to allow css targeting of sidebar open/closed state