Merge pull request #339 from ckeditor/i/353

Fix: Focus the editor before executing toolbar buttons' command. See ckeditor/ckeditor5#353.
ckeditor · Jan 28, 2020 · 5a700a2 · 5a700a2
2 parents 8f06626 + 8079b8f
commit 5a700a2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/imagestyle/imagestyleui.js b/src/imagestyle/imagestyleui.js
@@ -92,7 +92,10 @@ export default class ImageStyleUI extends Plugin {
 			view.bind( 'isEnabled' ).to( command, 'isEnabled' );
 			view.bind( 'isOn' ).to( command, 'value', value => value === style.name );
 
-			this.listenTo( view, 'execute', () => editor.execute( 'imageStyle', { value: style.name } ) );
+			this.listenTo( view, 'execute', () => {
+				editor.execute( 'imageStyle', { value: style.name } );
+				editor.editing.view.focus();
+			} );
 
 			return view;
 		} );

diff --git a/src/imagetextalternative/imagetextalternativeui.js b/src/imagetextalternative/imagetextalternativeui.js
@@ -78,7 +78,9 @@ export default class ImageTextAlternativeUI extends Plugin {
 
 			view.bind( 'isEnabled' ).to( command, 'isEnabled' );
 
-			this.listenTo( view, 'execute', () => this._showForm() );
+			this.listenTo( view, 'execute', () => {
+				this._showForm();
+			} );
 
 			return view;
 		} );