Skip to content

Commit

Permalink
Merge pull request #9896 from ckeditor/ck/9545-renaming-full-style
Browse files Browse the repository at this point in the history
Other (image): The default image style is now called `block`, instead of `full`. Its label reads now "Centered image" and it is represented by the appropriate icon in the image toolbar. See #9545.
  • Loading branch information
oleq authored Jun 21, 2021
2 parents 67b4c7d + f989da1 commit b5908e5
Show file tree
Hide file tree
Showing 95 changed files with 152 additions and 146 deletions.
2 changes: 1 addition & 1 deletion docs/_snippets/examples/multi-root-editor.js
Original file line number Diff line number Diff line change
Expand Up @@ -398,7 +398,7 @@ MultirootEditor
'insertTable', 'mediaEmbed', 'undo', 'redo' ],
image: {
toolbar: [
'imageStyle:inline', 'imageStyle:full', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative'
'imageStyle:inline', 'imageStyle:block', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative'
]
},
table: {
Expand Down
2 changes: 1 addition & 1 deletion docs/_snippets/features/toolbar-breakpoint.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ ClassicEditor
viewportTopOffset: window.getViewportTopOffsetConfig()
},
image: {
toolbar: [ 'imageStyle:inline', 'imageStyle:full', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:inline', 'imageStyle:block', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
},
codeBlock: {
languages: [
Expand Down
2 changes: 1 addition & 1 deletion docs/_snippets/features/toolbar-grouping.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ ClassicEditor
viewportTopOffset: window.getViewportTopOffsetConfig()
},
image: {
toolbar: [ 'imageStyle:inline', 'imageStyle:full', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:inline', 'imageStyle:block', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
},
codeBlock: {
languages: [
Expand Down
2 changes: 1 addition & 1 deletion docs/_snippets/features/toolbar-wrapping.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ ClassicEditor
viewportTopOffset: window.getViewportTopOffsetConfig()
},
image: {
toolbar: [ 'imageStyle:inline', 'imageStyle:full', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:inline', 'imageStyle:block', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
},
codeBlock: {
languages: [
Expand Down
2 changes: 1 addition & 1 deletion docs/builds/guides/development/dll-builds.md
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ For example:
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'toggleImageCaption',
Expand Down
4 changes: 2 additions & 2 deletions docs/builds/guides/integration/advanced-setup.md
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,7 @@ ClassicEditor.defaultConfig = {
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'toggleImageCaption',
Expand Down Expand Up @@ -354,7 +354,7 @@ ClassicEditor
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'toggleImageCaption',
Expand Down
2 changes: 1 addition & 1 deletion docs/builds/guides/integration/installing-plugins.md
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ ClassicEditor.defaultConfig = {
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'toggleImageCaption',
Expand Down
8 changes: 5 additions & 3 deletions docs/builds/guides/migration/migration-to-29.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,8 @@ Since the appearance of the image in the document depends on the image type (blo
* A few {@link module:image/imagestyle/utils#DEFAULT_DROPDOWN_DEFINITIONS default drop-downs} are provided.
* In the editor configuration {@link module:image/imagestyle/imagestyleui~ImageStyleDropdownDefinition a custom drop-down} can be declared.

* The name of the default block image style has changed from `full` to `block` (as the default style for the inline images is called `inline`), the default {@link builds/guides/integration/content-styles content styles} for these kind of images remains the same. The button label has also changed and now reads `Centered image`, so that it reflects the actual appearance of the image. If you customized the default appearance of the block images, you can change the button label by {@link module:image/image~ImageConfig#styles modifying an existing image style}.

* The format of the `config.image.styles` has changed. The list of the styles must be wrapped with the `options` array. Read more about the {@link module:image/image~ImageConfig#styles `image.styles` configuration}.

```js
Expand All @@ -59,7 +61,7 @@ Editor.create( document.querySelector( '#editor' ), {
...
image: {
styles: {
options: [ 'inline', 'full', 'side' ]
options: [ 'inline', 'block', 'side' ]
}
}
} );
Expand Down Expand Up @@ -117,7 +119,7 @@ Due to the changes mentioned above, the {@link module:image/image~ImageConfig#to
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'toggleImageCaption',
Expand All @@ -138,7 +140,7 @@ Due to the changes mentioned above, the {@link module:image/image~ImageConfig#to
'imageStyle:inline',
// A drop-down containing `alignLeft` and `alignRight` options
'imageStyle:wrapText',
// A drop-down containing `alignBlockLeft`, `full` (default) and `alignBlockRight` options
// A drop-down containing `alignBlockLeft`, `block` (default) and `alignBlockRight` options
'imageStyle:breakText'
]
}
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-autosave/src/autosave.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ import { debounce } from 'lodash-es';
* plugins: [ ArticlePluginSet, Autosave ],
* toolbar: [ 'heading', '|', 'bold', 'italic', 'link', 'bulletedList', 'numberedList', 'blockQuote', 'undo', 'redo' ],
* image: {
* toolbar: [ 'imageStyle:full', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ],
* toolbar: [ 'imageStyle:block', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ],
* },
* autosave: {
* save( editor ) {
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-autosave/tests/manual/autosave.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ ClassicEditor
plugins: [ ArticlePluginSet, Autosave ],
toolbar: [ 'heading', '|', 'bold', 'italic', 'link', 'bulletedList', 'numberedList', 'blockQuote', 'undo', 'redo' ],
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
}
} )
.then( editor => {
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-build-balloon-block/src/ckeditor.js
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ BalloonEditor.defaultConfig = {
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'toggleImageCaption',
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-build-balloon/src/ckeditor.js
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ BalloonEditor.defaultConfig = {
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'toggleImageCaption',
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-build-classic/src/ckeditor.js
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ ClassicEditor.defaultConfig = {
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'toggleImageCaption',
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-build-inline/src/ckeditor.js
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ InlineEditor.defaultConfig = {
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'toggleImageCaption',
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-ckfinder/tests/manual/ckfinder.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ ClassicEditor
plugins: [ ArticlePluginSet, ImageUpload, CKFinderUploadAdapter, CKFinder ],
toolbar: [ 'heading', '|', 'undo', 'redo', 'ckfinder' ],
image: {
toolbar: [ 'imageStyle:inline', 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:inline', 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
},
ckfinder: {
// eslint-disable-next-line max-len
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-core/src/editor/editor.js
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ export default class Editor {
* Stores all configurations specific to this editor instance.
*
* editor.config.get( 'image.toolbar' );
* // -> [ 'imageStyle:full', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
* // -> [ 'imageStyle:block', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
*
* @readonly
* @member {module:utils/config~Config}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ describe( 'ArticlePluginSet', () => {
editor = await ClassicTestEditor.create( editorElement, {
plugins: [ ArticlePluginSet ],
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side' ]
}
} );
} );
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-core/tests/manual/pendingactions.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ ClassicEditor
plugins: [ ArticlePluginSet, PendingActions ],
toolbar: [ 'heading', '|', 'bold', 'italic', 'link', 'bulletedList', 'numberedList', 'blockQuote', 'undo', 'redo' ],
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
}
} )
.then( editor => {
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-core/tests/manual/readonly.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ ClassicEditor
plugins: [ ArticlePluginSet, BalloonToolbar ],
toolbar: [ 'heading', '|', 'bold', 'italic', 'link', 'bulletedList', 'numberedList', 'blockQuote', 'undo', 'redo' ],
image: {
toolbar: [ 'imageStyle:inline', 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:inline', 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
},
balloonToolbar: [ 'bold', 'italic', 'link' ]
} )
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-core/tests/manual/version-collision.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ ClassicEditor
plugins: [ ArticlePluginSet ],
toolbar: [ 'heading', '|', 'bold', 'italic', 'link', 'bulletedList', 'numberedList', 'blockQuote', 'undo', 'redo' ],
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
}
} )
.then( editor => {
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-easy-image/tests/manual/easyimage.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ ClassicEditor
'redo'
],
image: {
toolbar: [ 'imageStyle:inline', 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:inline', 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
},
table: {
contentToolbar: [
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-editor-balloon/tests/ballooneditor.js
Original file line number Diff line number Diff line change
Expand Up @@ -373,7 +373,7 @@ describe( 'BalloonEditor', () => {
plugins: [ ArticlePluginSet ],
toolbar: [ 'heading', '|', 'bold', 'italic', 'link', 'bulletedList', 'numberedList', 'blockQuote' ],
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
}
} ) );
} );
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ function initEditors() {
plugins: [ ArticlePluginSet ],
toolbar: [ 'heading', '|', 'bold', 'italic', 'link', 'bulletedList', 'numberedList', 'blockQuote' ],
image: {
toolbar: [ 'imageStyle:inline', 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:inline', 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
}
} )
.then( editor => {
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-editor-balloon/tests/manual/memory.js
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ function initEditors() {
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'imageTextAlternative'
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-editor-classic/tests/classiceditor.js
Original file line number Diff line number Diff line change
Expand Up @@ -360,7 +360,7 @@ describe( 'ClassicEditor', () => {
plugins: [ ArticlePluginSet ],
toolbar: [ 'heading', '|', 'bold', 'italic', 'link', 'bulletedList', 'numberedList', 'blockQuote' ],
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
}
} ) );
} );
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-editor-classic/tests/manual/memory.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ function initEditor() {
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'imageTextAlternative'
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -414,7 +414,7 @@ describe( 'DecoupledEditor', () => {
plugins: [ ArticlePluginSet ],
toolbar: [ 'heading', '|', 'bold', 'italic', 'link', 'bulletedList', 'numberedList', 'blockQuote' ],
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
}
} ) );
} );
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-editor-decoupled/tests/manual/memory.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ function initEditor() {
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'imageTextAlternative'
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-editor-inline/tests/inlineeditor.js
Original file line number Diff line number Diff line change
Expand Up @@ -383,7 +383,7 @@ describe( 'InlineEditor', () => {
plugins: [ ArticlePluginSet ],
toolbar: [ 'heading', '|', 'bold', 'italic', 'link', 'bulletedList', 'numberedList', 'blockQuote' ],
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
}
} ) );
} );
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-editor-inline/tests/manual/memory.js
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ function initEditors() {
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'imageTextAlternative'
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -333,7 +333,7 @@ ClassicEditor
cloudServices: CS_CONFIG,
extraPlugins: [ ComplexBox ],
image: {
toolbar: [ 'imageStyle:inline', 'imageStyle:full', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:inline', 'imageStyle:block', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
},
table: {
contentToolbar: [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ ClassicEditor
cloudServices: CS_CONFIG,
extraPlugins: [ InfoBox ],
image: {
toolbar: [ 'imageStyle:inline', 'imageStyle:full', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:inline', 'imageStyle:block', 'imageStyle:side', '|', 'toggleImageCaption', 'imageTextAlternative' ]
},
table: {
contentToolbar: [
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-engine/tests/manual/slotconversion.js
Original file line number Diff line number Diff line change
Expand Up @@ -274,7 +274,7 @@ ClassicEditor
'redo'
],
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
},
table: {
contentToolbar: [
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-engine/tests/manual/tickets/1088/1.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ ClassicEditor
image: {
toolbar: [
'imageStyle:inline',
'imageStyle:full',
'imageStyle:block',
'imageStyle:side',
'|',
'imageTextAlternative'
Expand Down
8 changes: 4 additions & 4 deletions packages/ckeditor5-font/tests/integration.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ describe( 'Integration test Font', () => {
.create( element, {
plugins: [ Font, ArticlePluginSet ],
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side' ]
}
} )
.then( newEditor => {
Expand Down Expand Up @@ -72,7 +72,7 @@ describe( 'Integration test Font', () => {
supportAllValues: true
},
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side' ]
}
} )
.then( editor => {
Expand Down Expand Up @@ -118,7 +118,7 @@ describe( 'Integration test Font', () => {
supportAllValues: true
},
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side' ]
}
} )
.then( editor => {
Expand Down Expand Up @@ -182,7 +182,7 @@ describe( 'Integration test Font', () => {
supportAllValues: true
},
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side' ]
}
} )
.then( editor => {
Expand Down
2 changes: 1 addition & 1 deletion packages/ckeditor5-html-embed/tests/manual/htmlembed.js
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ async function reloadEditor( config = {} ) {
'undo', 'redo', '|', 'htmlEmbed', 'mediaEmbed'
],
image: {
toolbar: [ 'imageStyle:full', 'imageStyle:side', '|', 'imageTextAlternative' ]
toolbar: [ 'imageStyle:block', 'imageStyle:side', '|', 'imageTextAlternative' ]
}
};

Expand Down
Loading

0 comments on commit b5908e5

Please sign in to comment.