From df0ceb17e83e5b5eec980fc30e830d0416f19b1f Mon Sep 17 00:00:00 2001
From: howard-stearns <howard.stearns@gmail.com>
Date: Fri, 12 Aug 2016 11:09:22 -0700
Subject: [PATCH 1/4] send (hi res image) details as a json string

---
 interface/src/ui/Snapshot.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/interface/src/ui/Snapshot.cpp b/interface/src/ui/Snapshot.cpp
index beb86af1c4..b7cdb1a126 100644
--- a/interface/src/ui/Snapshot.cpp
+++ b/interface/src/ui/Snapshot.cpp
@@ -197,7 +197,8 @@ void SnapshotUploader::uploadSuccess(QNetworkReply& reply) {
         QJsonObject userStoryObject;
         QJsonObject detailsObject;
         detailsObject.insert("image_url", imageUrl);
-        userStoryObject.insert("details", detailsObject);
+        QString pickledDetails = QJsonDocument(detailsObject).toJson();
+        userStoryObject.insert("details", pickledDetails);
         userStoryObject.insert("thumbnail_url", thumbnailUrl);
         userStoryObject.insert("place_name", placeName);
         userStoryObject.insert("path", currentPath);

From ae9af88db287df65ad6d61cb24b2d29d84b996f8 Mon Sep 17 00:00:00 2001
From: howard-stearns <howard.stearns@gmail.com>
Date: Fri, 12 Aug 2016 11:28:35 -0700
Subject: [PATCH 2/4] Spacing and better size for HMD pictures.

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

diff --git a/scripts/system/html/ShareSnapshot.html b/scripts/system/html/ShareSnapshot.html
index 00f805728c..a30532bbc9 100644
--- a/scripts/system/html/ShareSnapshot.html
+++ b/scripts/system/html/ShareSnapshot.html
@@ -35,11 +35,13 @@
       }
 
       .snapsection{
-      width:95% !important;
-      padding-right:0px;
+        width:95% !important;
+        padding-right:0px;
+      }
+      div#sharing {
+        margin-top:50px;
       }
 
-    div.clear        { clear: both; }
     </style>
     <script type="text/javascript" src="qrc:///qtwebchannel/qwebchannel.js"></script>
     <script type="text/javascript" src="js/eventBridgeLoader.js"></script>
diff --git a/scripts/system/snapshot.js b/scripts/system/snapshot.js
index 4c0ba37ab3..39bcc7fe90 100644
--- a/scripts/system/snapshot.js
+++ b/scripts/system/snapshot.js
@@ -21,7 +21,7 @@ var button = toolBar.addButton({
 });
 
 function confirmShare(data) {
-    var dialog = new OverlayWebWindow('Snapshot', Script.resolvePath("html/ShareSnapshot.html"), 800, 300);
+    var dialog = new OverlayWebWindow('Snapshot', Script.resolvePath("html/ShareSnapshot.html"), 800, 470);
     function onMessage(message) {
         if (message == 'ready') { // The window can now receive data from us.
             dialog.emitScriptEvent(data); // Send it.

From 1b62332058079cd0779657de3a12316b8b92688c Mon Sep 17 00:00:00 2001
From: howard-stearns <howard.stearns@gmail.com>
Date: Fri, 12 Aug 2016 11:50:11 -0700
Subject: [PATCH 3/4] don't show checkboxes on confirmation dialog when there's
 only one picture

---
 scripts/system/html/ShareSnapshot.html | 23 +++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git a/scripts/system/html/ShareSnapshot.html b/scripts/system/html/ShareSnapshot.html
index a30532bbc9..f9cc959ba9 100644
--- a/scripts/system/html/ShareSnapshot.html
+++ b/scripts/system/html/ShareSnapshot.html
@@ -46,7 +46,7 @@
     <script type="text/javascript" src="qrc:///qtwebchannel/qwebchannel.js"></script>
     <script type="text/javascript" src="js/eventBridgeLoader.js"></script>
     <script>
-    var paths = [], idCounter = 0;
+    var paths = [], idCounter = 0, useCheckboxes;
     function addImage(data) {
         var div = document.createElement("DIV"),
             input = document.createElement("INPUT"),
@@ -56,15 +56,17 @@
         function toggle() { data.share = input.checked; }
         img.src = data.localPath;
         div.appendChild(img);
-        // Our stylesheet(?) requires input.id to match label.for. Otherwise input doesn't display the check state.
-        label.setAttribute('for', id); // cannot do label.for =
-        input.id = id;
-        input.type = "checkbox";
-        input.checked = data.share = true;
-        input.addEventListener('change', toggle);
-        div.class = "property checkbox";
-        div.appendChild(input);
-        div.appendChild(label);
+        if (useCheckboxes) { // I'd rather use css, but the included stylesheet is quite particular.
+            // Our stylesheet(?) requires input.id to match label.for. Otherwise input doesn't display the check state.
+            label.setAttribute('for', id); // cannot do label.for =
+            input.id = id;
+            input.type = "checkbox";
+            input.checked = data.share = true;
+            input.addEventListener('change', toggle);
+            div.class = "property checkbox";
+            div.appendChild(input);
+            div.appendChild(label);
+        }
         document.getElementById("snapshot-images").appendChild(div);
         paths.push(data);
 
@@ -93,6 +95,7 @@
                 handleShareButtons(shareMsg);
                 
                 // rest are image paths which we add
+                useCheckboxes = message.length > 1;
                 message.forEach(addImage);
             });
             EventBridge.emitWebEvent('ready');

From afe376a0a26c6859301a404c3cd44eb265486308 Mon Sep 17 00:00:00 2001
From: howard-stearns <howard.stearns@gmail.com>
Date: Fri, 12 Aug 2016 14:08:50 -0700
Subject: [PATCH 4/4] fix breakage I made to snapshot uploading

---
 scripts/system/html/ShareSnapshot.html | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/scripts/system/html/ShareSnapshot.html b/scripts/system/html/ShareSnapshot.html
index f9cc959ba9..7754371547 100644
--- a/scripts/system/html/ShareSnapshot.html
+++ b/scripts/system/html/ShareSnapshot.html
@@ -56,12 +56,13 @@
         function toggle() { data.share = input.checked; }
         img.src = data.localPath;
         div.appendChild(img);
+        data.share = true;
         if (useCheckboxes) { // I'd rather use css, but the included stylesheet is quite particular.
             // Our stylesheet(?) requires input.id to match label.for. Otherwise input doesn't display the check state.
             label.setAttribute('for', id); // cannot do label.for =
             input.id = id;
             input.type = "checkbox";
-            input.checked = data.share = true;
+            input.checked = true;
             input.addEventListener('change', toggle);
             div.class = "property checkbox";
             div.appendChild(input);