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
Extract dropdown #3443
Extract dropdown #3443
Conversation
zepfietje
commented
Aug 9, 2022
•
edited
edited
- Update docs about configuring scripts
- Dropdown panel width
- Dropdown border opacity
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
2379e73
to
6866d4c
Compare
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
Merge conflicts |
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.
You will need to revert all asset changes before I can merge this
Yeah, so want to keep Alpine Floating UI shipped with the forms package? And what about the others, should they have docs to manually add it, since it wasn't there now? |
Floating UI is used by the forms package Tables and admin require the forms package already So no extra docs needed, just keep this PR as a refactor to a common dropdown Blade component. |
I've got a separate PR to add action groups support to the notifications package. So how should that be handled then? |
Resolved, @danharrin! |