-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Allow a more flexible toolbar configuration in the dropdown #2602
Comments
Separating pens and markers 👍 👍 👍 |
The second one would be more precise as we can have either I think that the latter option would be less confusing as you could just omit The question is how to deal with missing |
Like how to determine the default value of |
There has been no activity on this issue for the past year. We've marked it as stale and will close it in 30 days. We understand it may be relevant, so if you're interested in the solution, leave a comment or reaction under this issue. |
There has been no activity on this issue for the past year. We've marked it as stale and will close it in 30 days. We understand it may still be relevant, so if you're interested in the solution, leave a comment or reaction under this issue. |
We've closed your issue due to inactivity over the last year. We understand that the issue may still be relevant. If so, feel free to open a new one (and link this issue to it). |
A follow-up of ckeditor/ckeditor5-highlight#6 (comment). Quoting myself:
A mixed-up approach
A separate property approach
It makes more sense to me because it doesn't mix up the content logic and the UI.
In such situation, the order in
options
doesn't matter. We also provide a defaulttoolbar
layout, when no specified (and the default istoolbar: [ all options, '|', 'removeHighlight' ]
)The text was updated successfully, but these errors were encountered: