浏览代码

Fix: Wrong toolbar position when config.toolbar.viewportTopOffset is not specified.

Aleksander Nowodzinski 8 年之前
父节点
当前提交
53dc796c37

+ 1 - 1
packages/ckeditor5-editor-classic/src/classiceditorui.js

@@ -61,7 +61,7 @@ export default class ClassicEditorUI {
 		view.toolbar.bind( 'isActive' ).to( this.focusTracker, 'isFocused' );
 		view.toolbar.limiterElement = view.element;
 
-		if ( this._toolbarConfig ) {
+		if ( this._toolbarConfig && this._toolbarConfig.viewportTopOffset ) {
 			view.toolbar.viewportTopOffset = this._toolbarConfig.viewportTopOffset;
 		}
 

+ 5 - 1
packages/ckeditor5-editor-classic/tests/classiceditorui.js

@@ -87,7 +87,11 @@ describe( 'ClassicEditorUI', () => {
 				expect( view.toolbar.limiterElement ).to.equal( view.element );
 			} );
 
-			it( 'sets view.toolbar#viewportTopOffset', () => {
+			it( 'doesn\'t set view.toolbar#viewportTopOffset, if not specified in the config', () => {
+				expect( view.toolbar.viewportTopOffset ).to.equal( 0 );
+			} );
+
+			it( 'sets view.toolbar#viewportTopOffset, when specified in the config', () => {
 				editorElement = document.createElement( 'div' );
 				document.body.appendChild( editorElement );