Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Commit

Permalink
Merge pull request #105 from ckeditor/t/104
Browse files Browse the repository at this point in the history
Internal: Aligned the codebase after refactoring of the `ck-editor-toolbar` mixin in theme-lark. Closes #104.
  • Loading branch information
oskarwrobel authored Apr 25, 2017
2 parents 4db7b34 + 69bb99a commit 862244c
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 1 deletion.
9 changes: 8 additions & 1 deletion src/imagetoolbar.js
Original file line number Diff line number Diff line change
Expand Up @@ -59,11 +59,18 @@ export default class ImageToolbar extends Plugin {
const promises = [];
const toolbar = new ToolbarView();

// Add CSS class to the toolbar.
Template.extend( toolbar.template, {
attributes: {
class: 'ck-editor-toolbar'
}
} );

// Add CSS class to the panel.
Template.extend( panel.template, {
attributes: {
class: [
'ck-toolbar__container'
'ck-toolbar-container'
]
}
} );
Expand Down
5 changes: 5 additions & 0 deletions tests/imagetoolbar.js
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,11 @@ describe( 'ImageToolbar', () => {
} );
} );

it( 'should set css classes', () => {
expect( panel.element.classList.contains( 'ck-toolbar-container' ) ).to.be.true;
expect( panel.content.get( 0 ).element.classList.contains( 'ck-editor-toolbar' ) ).to.be.true;
} );

it( 'should add ImageBalloonPanel to view body', () => {
expect( panel ).to.be.instanceOf( ImageBalloonPanel );
} );
Expand Down

0 comments on commit 862244c

Please sign in to comment.