瀏覽代碼

TreeView.createRoot() should return the created view element. Fixes #280.

Piotrek Koszuliński 9 年之前
父節點
當前提交
8e64334d83

+ 3 - 0
packages/ckeditor5-engine/src/treeview/treeview.js

@@ -118,6 +118,7 @@ export default class TreeView {
 	 *
 	 * @param {HTMLElement} domRoot DOM element in which the tree view should do change.
 	 * @param {String} name Name of the root.
+	 * @returns {core.treeView.element} The created view root element.
 	 */
 	createRoot( domRoot, name ) {
 		const viewRoot = this.domConverter.domToView( domRoot, { bind: true, withChildren: false } );
@@ -135,6 +136,8 @@ export default class TreeView {
 		for ( let observer of this._observers ) {
 			observer.observe( domRoot, name );
 		}
+
+		return viewRoot;
 	}
 
 	/**

+ 3 - 1
packages/ckeditor5-engine/tests/treeview/treeview.js

@@ -39,7 +39,7 @@ describe( 'TreeView', () => {
 			domDiv.appendChild( domP );
 
 			const treeView = new TreeView();
-			treeView.createRoot( domDiv, 'editor' );
+			const ret = treeView.createRoot( domDiv, 'editor' );
 
 			expect( count( treeView.domRoots ) ).to.equal( 1 );
 			expect( count( treeView.viewRoots ) ).to.equal( 1 );
@@ -47,6 +47,8 @@ describe( 'TreeView', () => {
 			const domRoot = treeView.domRoots.get( 'editor' );
 			const viewRoot = treeView.viewRoots.get( 'editor' );
 
+			expect( ret ).to.equal( viewRoot );
+
 			expect( domRoot ).to.equal( domDiv );
 			expect( treeView.domConverter.getCorrespondingDom( viewRoot ) ).to.equal( domDiv );
 			expect( viewRoot.name ).to.equal( 'div' );