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.

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

Builds should not define a shortened config version #572

Closed
Reinmar opened this issue Sep 30, 2017 · 2 comments
Closed

Builds should not define a shortened config version #572

Reinmar opened this issue Sep 30, 2017 · 2 comments
Assignees
Labels
type:bug This issue reports a buggy (incorrect) behavior.
Milestone

Comments

@Reinmar
Copy link
Member

Reinmar commented Sep 30, 2017

I tried to create an editor (using the classic build):

return ClassicEditor
	.create( document.querySelector( '#snippet-classic-editor' ), {
		toolbar: {
			viewportTopOffset: 60
		}
	} )

Which didn't work well because no toolbar items were rendered. This is because the build specifies config.toolbar = [ 'item1', 'item2' ... ] so setting that to the object format overrides the value completely.

Instead, if the build will configure config.toolbar.items = [ ... ] everything will work fine.

@Reinmar Reinmar added status:confirmed type:bug This issue reports a buggy (incorrect) behavior. labels Sep 30, 2017
@Reinmar Reinmar added this to the iteration 12 milestone Sep 30, 2017
@Reinmar Reinmar self-assigned this Sep 30, 2017
Reinmar added a commit to ckeditor/ckeditor5-build-classic that referenced this issue Sep 30, 2017
@Reinmar
Copy link
Member Author

Reinmar commented Sep 30, 2017

Of course, if we configure toolbar items through config.toolbar.items one can still override them easily through config.toolbar (using the simplified format).

Reinmar added a commit to ckeditor/ckeditor5-build-classic that referenced this issue Sep 30, 2017
Reinmar added a commit to ckeditor/ckeditor5-build-balloon that referenced this issue Sep 30, 2017
Reinmar added a commit to ckeditor/ckeditor5-build-inline that referenced this issue Sep 30, 2017
@Reinmar
Copy link
Member Author

Reinmar commented Sep 30, 2017

@Reinmar Reinmar closed this as completed Sep 30, 2017
JDinABox pushed a commit to JDinABox/ckeditor5-build-markdown that referenced this issue Sep 6, 2021
rivernews pushed a commit to rivernews/ckeditor5-build-balloon-2022 that referenced this issue Dec 31, 2021
fl01337 pushed a commit to fl01337/ckeditor-5-build-inline that referenced this issue May 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

No branches or pull requests

1 participant