Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Allow array value in config.removeButtons #5122

Closed
Comandeer opened this issue Mar 12, 2022 · 1 comment · Fixed by #5302
Closed

Allow array value in config.removeButtons #5122

Comandeer opened this issue Mar 12, 2022 · 1 comment · Fixed by #5302
Assignees
@Comandeer
Labels
core The issue is caused by the editor core code. status:confirmed An issue confirmed by the development team. type:feature A feature request.
Milestone
4.20.0

Comments

@Comandeer
Copy link
Member

Type of report

Feature request

Provide description of the new feature

The config.removePlugins supports passing plugins' names as an array, however config.removeButtons supports only passing buttons' names as a string. We could allow the same syntax for both options.

We can also think about this improvement for all other config.remove* options.
@Comandeer Comandeer added type:feature A feature request. status:confirmed An issue confirmed by the development team. core The issue is caused by the editor core code. size:? labels Mar 12, 2022
@Comandeer Comandeer self-assigned this Aug 1, 2022
@Comandeer Comandeer mentioned this issue Aug 1, 2022
Merged
3 tasks
@CKEditorBot
Copy link
Collaborator

Closed in #5302

@CKEditorBot CKEditorBot added this to the 4.20.0 milestone Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Comandeer Comandeer

Labels
core The issue is caused by the editor core code. status:confirmed An issue confirmed by the development team. type:feature A feature request.
Projects
None yet
Milestone
4.20.0
Development

Successfully merging a pull request may close this issue.

Add support for array in the config.removeButtons ckeditor/ckeditor4
3 participants
@Comandeer @jacekbogdanski @CKEditorBot