Browse Source

Added missing comments.

Aleksander Nowodzinski 10 years ago
parent
commit
24c0d67361

+ 2 - 0
packages/ckeditor5-ui/src/ui/controller.js

@@ -56,6 +56,8 @@ CKEDITOR.define( [ 'collection', 'model' ], function( Collection, Model ) {
 		 * @returns
 		 */
 		destroy() {
+			// Note: Because this.view.destroy() can by sync as well as async,
+			// it is wrapped in promise.
 			return Promise.resolve()
 				.then( () => {
 					return this.view.destroy();

+ 4 - 0
packages/ckeditor5-ui/src/ui/view.js

@@ -268,6 +268,10 @@ CKEDITOR.define( [
 			}
 		}
 
+		/**
+		 * @param
+		 * @returns
+		 */
 		append( view, regionName ) {
 			this.regions.get( regionName ).append( view );
 		}