Added some documentation

dev/Knoin/AbstractViews.js

@@ -29,8 +29,8 @@ class AbstractView {
 
 /*
 	onBuild() {}
-	onBeforeShow() {}
+	onBeforeShow() {} // Happens before: hidden = false
-	onShow() {}
+	onShow() {}       // Happens after: hidden = false
 	onHide() {}
 */
 
@@ -71,13 +71,17 @@ export class AbstractViewPopup extends AbstractView
 		shortcuts.add('backspace', '', name, inFocus());
 	}
 
+	// Happens when user hits Escape key
 	onClose() {
 		return true;
 	}
 
 /*
-	afterShow() {}
+	onBeforeShow() {} // Happens before showModal()
-	afterHide() {}
+	onShow() {}       // Happens after showModal()
+	afterShow() {}    // Happens after showModal() animation transitionend
+	onHide() {}       // Happens before animation transitionend
+	afterHide() {}    // Happens after  animation transitionend
 
 	closeCommand() {}
 */

dev/Knoin/Knoin.js

@@ -103,17 +103,6 @@ const
 							vmDom.classList.remove('animate'); // trigger the transitions
 						}
 						arePopupsVisible(0 < visiblePopups.size);
-/*
-						// the old ko.bindingHandlers.modal
-						const close = vmDom.querySelector('.close'),
-							click = () => vm.modalVisibility(false);
-						if (close) {
-							close.addEventListener('click.koModal', click);
-							ko.utils.domNodeDisposal.addDisposeCallback(vmDom, () =>
-								close.removeEventListener('click.koModal', click)
-							);
-						}
-*/
 					});
 					vmDom.addEventListener('transitionend', endShowHide);
 				}