Browse Source

Removed iconManager property from EditorUI Controller.

Aleksander Nowodzinski 9 years ago
parent
commit
369c9438c3

+ 3 - 9
packages/ckeditor5-ui/src/editorui/editorui.js

@@ -72,14 +72,6 @@ export default class EditorUI extends Controller {
 	 */
 	_setupIconManager() {
 		/**
-		 * A reference to IconManager controller.
-		 *
-		 * @readonly
-		 * @member {ui.Controller} ui.editorUI.EditorUI#iconManager
-		 */
-		this.iconManager = new Controller( iconManagerModel, new IconManagerView( iconManagerModel ) );
-
-		/**
 		 * Icons available in the UI.
 		 *
 		 * @readonly
@@ -87,7 +79,9 @@ export default class EditorUI extends Controller {
 		 */
 		this.icons = iconManagerModel.icons;
 
-		this.collections.get( 'body' ).add( this.iconManager );
+		this.collections.get( 'body' ).add(
+			new Controller( iconManagerModel, new IconManagerView( iconManagerModel ) )
+		);
 	}
 }
 

+ 0 - 6
packages/ckeditor5-ui/tests/editorui/editorui.js

@@ -47,12 +47,6 @@ describe( 'EditorUI', () => {
 	} );
 
 	describe( '_setupIconManager', () => {
-		it( 'initializes icon manager and sets "iconManager" property', () => {
-			editorUI._setupIconManager();
-
-			expect( editorUI ).to.have.property( 'iconManager' );
-		} );
-
 		it( 'sets "icon" property', () => {
 			editorUI._setupIconManager();