-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
List: Add inspector interface to choose bullet or number type #51480
Comments
Good thoughts. I actually think both can work, but also that it's one of those issues that can be hashed out in implementation. |
It seems like we should be able to copy over that same list item inspector control to a typography panel for the categories list block, yes. All the more reason that it's good for it to be compact. |
All of this should also be applied to the details block too :D |
Do you mean in terms of changing the chevron? I'd love to provide options there, but we always get stuck at limitations in the base element. Are there new options for changing the icon? |
It's certainly limited, but some CSS can be used. This is what we did in TT4, not the prettiest thing but I wanted to avoid too much CSS. |
Nice, that works. It doesn't animate, though, and remains somewhat non-standard, so I wonder if the same interface will work here 🤔 |
Using pseudo-elements will give us more control than that, but I didn't want to overcomplicate things in the theme, that's why I quickly came here to mention we could support that better from the block directly instead |
#51186 adds an option to choose number types in the inspector. While the feature can land soon, we have an opportunity here to improve the interface in a few ways:
In addition to that, we can make likely make the control into a style control, so that it doesn't split the inspector by adding just a single panel. All of this we can likely accomplish with a flyout menu pattern, like so, shown here for unordered list:
Similarly we can apply the same pattern for ordered lists:
What do you think?
Related: a similar flyout interface for the Paragraph block in #51481.
The text was updated successfully, but these errors were encountered: