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 #28 from ckeditor/t/27
Browse files Browse the repository at this point in the history
Internal: Simplify view property in EditorUI. Closes #27.
  • Loading branch information
Piotr Jasiun committed Jan 24, 2019
2 parents 3f3c191 + 180ada4 commit 2b861b9
Showing 1 changed file with 4 additions and 14 deletions.
18 changes: 4 additions & 14 deletions src/decouplededitorui.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,10 @@ export default class DecoupledEditorUI extends EditorUI {
/**
* The main (top–most) view of the editor UI.
*
* @private
* @member {module:ui/editorui/editoruiview~EditorUIView} #_view
* @readonly
* @member {module:ui/editorui/editoruiview~EditorUIView} #view
*/
this._view = view;
this.view = view;

/**
* A normalized `config.toolbar` object.
Expand All @@ -43,16 +43,6 @@ export default class DecoupledEditorUI extends EditorUI {
this._toolbarConfig = normalizeToolbarConfig( editor.config.get( 'toolbar' ) );
}

/**
* The main (top–most) view of the editor UI.
*
* @readonly
* @member {module:ui/editorui/editoruiview~EditorUIView} #view
*/
get view() {
return this._view;
}

/**
* Initializes the UI.
*/
Expand Down Expand Up @@ -89,7 +79,7 @@ export default class DecoupledEditorUI extends EditorUI {
* @inheritDoc
*/
destroy() {
this._view.destroy();
this.view.destroy();

super.destroy();
}
Expand Down

0 comments on commit 2b861b9

Please sign in to comment.