From 1d01f0c13dab3a9e42a0b5f4e43392b968065415 Mon Sep 17 00:00:00 2001
From: Jake <jake.read@cba.mit.edu>
Date: Mon, 26 Nov 2018 10:27:33 -0500
Subject: [PATCH] comments

---
 client/ui/uiButton.js | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/client/ui/uiButton.js b/client/ui/uiButton.js
index 955a7da..3d8f021 100644
--- a/client/ui/uiButton.js
+++ b/client/ui/uiButton.js
@@ -1,8 +1,10 @@
 (function(){
-	// client-side button object 
+	/* KRITICAL STEP */
+	// this is our object, that we will later append
+	// to the .lump position 
 	var btn = {}
 
-	// make the dom object 
+	// our dom element
 	var li = document.createElement('li')
 	li.innerHTML = 'button test'
 	li.addEventListener('click', function(evt){
@@ -15,9 +17,17 @@
 		socketSend('put ui change', data)
 	})
 
+	/* KRITICAL STEP */
+	// we have to give our 'lump' object a .domElem, this
+	// will be referenced upstream to append to the right module 
 	// append it to the dom so that it can be appended on init 
 	btn.domElem = li 
 
+	/* KRITICAL STEP */
+	// whatever is posted to .lump will receive messages through
+	// .onMessage, so if we don't write one, we'll cause errors
+	// upstream, and besides, wouldn't be able to get anything from
+	// the server 
 	btn.onMessage = function(msg){
 		console.log('got message in client side ui object', msg)
 		if(msg.call == 'setText'){
@@ -25,8 +35,10 @@
 		}
 	}
 
+	/* KRITICAL STEP */
 	// expect this to only be used once
 	// it's basically our init function 
+	// and gets called once the module is loaded 
 	window.registerNewModule = function(id, key){
 		btn.parentId = id 
 		btn.key = key 
-- 
GitLab