From d7348aabc882e6284c9eeb77b86d00d2444165c7 Mon Sep 17 00:00:00 2001
From: Zach Fox <fox@highfidelity.io>
Date: Tue, 25 Apr 2017 10:10:58 -0700
Subject: [PATCH] Quick comment updates

---
 scripts/system/html/js/SnapshotReview.js | 6 +++---
 scripts/system/snapshot.js               | 4 ++--
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/scripts/system/html/js/SnapshotReview.js b/scripts/system/html/js/SnapshotReview.js
index 4c5419869d..c3fb20a2f9 100644
--- a/scripts/system/html/js/SnapshotReview.js
+++ b/scripts/system/html/js/SnapshotReview.js
@@ -131,7 +131,7 @@ function createShareOverlay(parentID, isGif, shareURL, hifiShareButtonsDisabled)
     return shareOverlayContainer;
 }
 function selectImageToShare(selectedID) {
-    selectedID = selectedID.id; // Why is this necessary?
+    selectedID = selectedID.id; // `selectedID` is passed as an HTML object to these functions; we just want the ID
     var shareOverlayContainer = document.getElementById(selectedID + "shareOverlayContainer");
     var shareBar = document.getElementById(selectedID + "shareBar");
     var shareOverlayBackground = document.getElementById(selectedID + "shareOverlayBackground");
@@ -153,7 +153,7 @@ function shareForUrl(selectedID) {
     }));
 }
 function shareWithEveryone(selectedID, isGif) {
-    selectedID = selectedID.id; // Why is this necessary?
+    selectedID = selectedID.id; // `selectedID` is passed as an HTML object to these functions; we just want the ID
 
     document.getElementById(selectedID + "shareWithEveryoneButton").setAttribute("disabled", "disabled");
     document.getElementById(selectedID + "inviteConnectionsCheckbox").setAttribute("disabled", "disabled");
@@ -174,7 +174,7 @@ function shareButtonClicked(selectedID) {
     }));
 }
 function cancelSharing(selectedID) {
-    selectedID = selectedID.id; // Why is this necessary?
+    selectedID = selectedID.id; // `selectedID` is passed as an HTML object to these functions; we just want the ID
     var shareOverlayContainer = document.getElementById(selectedID + "shareOverlayContainer");
     var shareBar = document.getElementById(selectedID + "shareBar");
     var shareOverlayBackground = document.getElementById(selectedID + "shareOverlayBackground");
diff --git a/scripts/system/snapshot.js b/scripts/system/snapshot.js
index 0ac809f8a7..fe6ab3f28f 100644
--- a/scripts/system/snapshot.js
+++ b/scripts/system/snapshot.js
@@ -98,7 +98,7 @@ function onMessage(message) {
     var isLoggedIn;
     var needsLogin = false;
     switch (message.action) {
-        case 'ready':  // Send it.
+        case 'ready': // DOM is ready and page has loaded
             tablet.emitScriptEvent(JSON.stringify({
                 type: "snapshot",
                 action: "captureSettings",
@@ -137,7 +137,7 @@ function onMessage(message) {
                 print('Sharing snapshot with audience "for_url":', message.data);
                 Window.shareSnapshot(message.data, message.href || href);
             } else {
-                // TODO?
+                // TODO
             }
             break;
         case 'shareSnapshotWithEveryone':