-
Notifications
You must be signed in to change notification settings - Fork 12
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
Multiple level dropdowns #194
Conversation
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.
@ebrelsford I've got the list items to truncate and made labels not go over their container (and also adjusted whether it's a drop-down or drop-up based on screen position) but I'm having trouble getting the toggle itself to use truncation due to how its implemented in the library. It's basically a div with an |
@ebrelsford nevermind the above, I figured out how to make the ellipsis work correctly here! |
Description
This PR enables style presets to have a second nested level to help organize long lists of styles. We do not allow more than two levels of nesting.
TODO
I'm debating if it makes sense to apply a similar treatment here, always treat styles for a branch pattern as a multi-dropdown, or do nothingbranchPatterns
Closes #193
QA steps
local.js
config to include style presets like the following:sveltestrap
library for consistency)Author checklist
Create the PR
After approval
main
into your branch, resolve any conflictsmain