From f020bd9129bf70e998e0ae206218493a72ebfa81 Mon Sep 17 00:00:00 2001
From: Prateek Saxena <prtksxna@gmail.com>
Date: Sat, 23 Apr 2016 11:10:17 +0530
Subject: Add a way to remove image from post once added

Accidents happen, you shouldn't have to refresh the entire page
to get rid of the image.
---
 views/new-post.php | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

diff --git a/views/new-post.php b/views/new-post.php
index 57b7521..2376431 100644
--- a/views/new-post.php
+++ b/views/new-post.php
@@ -28,7 +28,10 @@
           <label for="note_photo"><code>photo</code></label>
           <input type="file" name="note_photo" id="note_photo" accept="image/*" onchange="previewPhoto(event)">
           <br>
-          <img src="" id="photo_preview" style="max-width: 300px; max-height: 300px;">
+          <div id="photo_preview_container">
+            <img src="" id="photo_preview" style="max-width: 300px; max-height: 300px;">
+            <button type="button" class="btn btn-danger btn-sm" id="remove_photo"><span class="glyphicon glyphicon-trash" aria-hidden="true"></span> Remove image</button>
+          </div>
         </div>
 
         <div class="form-group">
@@ -133,15 +136,20 @@
 
 <script>
 function previewPhoto(event) {
-  $("#photo_preview")
-    .show()
-    .attr("src", URL.createObjectURL(event.target.files[0]) );
+  $("#photo_preview_container").show();
+  $("#photo_preview").attr("src", URL.createObjectURL(event.target.files[0]) );
 }
 
 $(function(){
 
   var userHasSetCategory = false;
-  $("#photo_preview").hide();
+
+  $("#photo_preview_container").hide();
+  $("#remove_photo").on("click", function(){
+    $("#note_photo").val("");
+    $("#photo_preview").attr("src", "" );
+    $("#photo_preview_container").hide();
+  });
 
   $("#note_content").on('change keyup', function(e){
     var text = $("#note_content").val();
-- 
cgit v1.2.3


From e42a94a988e261c0b49022c8cba6a321878e7299 Mon Sep 17 00:00:00 2001
From: Prateek Saxena <prtksxna@gmail.com>
Date: Sat, 23 Apr 2016 11:14:14 +0530
Subject: Use jQuery event handler instead of the HTML onchange attribute

The `#note_photo` form element is the only instance in the application
that used the attribute.
---
 views/new-post.php | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/views/new-post.php b/views/new-post.php
index 2376431..3ff8951 100644
--- a/views/new-post.php
+++ b/views/new-post.php
@@ -26,7 +26,7 @@
 
         <div class="form-group">
           <label for="note_photo"><code>photo</code></label>
-          <input type="file" name="note_photo" id="note_photo" accept="image/*" onchange="previewPhoto(event)">
+          <input type="file" name="note_photo" id="note_photo" accept="image/*">
           <br>
           <div id="photo_preview_container">
             <img src="" id="photo_preview" style="max-width: 300px; max-height: 300px;">
@@ -135,16 +135,15 @@
 </style>
 
 <script>
-function previewPhoto(event) {
-  $("#photo_preview_container").show();
-  $("#photo_preview").attr("src", URL.createObjectURL(event.target.files[0]) );
-}
-
 $(function(){
 
   var userHasSetCategory = false;
 
   $("#photo_preview_container").hide();
+  $("#note_photo").on("change", function(e){
+    $("#photo_preview_container").show();
+    $("#photo_preview").attr("src", URL.createObjectURL(e.target.files[0]) );
+  });
   $("#remove_photo").on("click", function(){
     $("#note_photo").val("");
     $("#photo_preview").attr("src", "" );
-- 
cgit v1.2.3


From 3df5df9dc46d88efdc94e684a8af9c60d39250ee Mon Sep 17 00:00:00 2001
From: Prateek Saxena <prtksxna@gmail.com>
Date: Sun, 24 Apr 2016 12:41:17 +0530
Subject: Add 196px icon for Android Chrome

Firefox picks up the last defined icon PNG for the add to
homescreen feature - https://bugzilla.mozilla.org/751712
---
 public/images/quill-icon-196.png | Bin 0 -> 11944 bytes
 views/layout.php                 |   1 +
 2 files changed, 1 insertion(+)
 create mode 100644 public/images/quill-icon-196.png

diff --git a/public/images/quill-icon-196.png b/public/images/quill-icon-196.png
new file mode 100644
index 0000000..1d6c564
Binary files /dev/null and b/public/images/quill-icon-196.png differ
diff --git a/views/layout.php b/views/layout.php
index da97687..bef6f3b 100644
--- a/views/layout.php
+++ b/views/layout.php
@@ -26,6 +26,7 @@
     <link rel="apple-touch-icon" sizes="144x144" href="/images/quill-icon-144.png">
 
     <link rel="icon" href="/favicon.ico" type="image/x-icon">
+    <link rel="icon" sizes="196x196" href="/images/quill-icon-196.png">
 
     <script src="/js/jquery-1.7.1.min.js"></script>
     <script src="/js/script.js"></script>
-- 
cgit v1.2.3


From 3f7f28418f075756f027a0a268ae78fbf6003aa0 Mon Sep 17 00:00:00 2001
From: Prateek Saxena <prtksxna@gmail.com>
Date: Sun, 24 Apr 2016 12:49:11 +0530
Subject: Add fullscreen meta tags for Safari and Chrome

Also set the theme color for Chrome to the blue used
in the logo.
---
 views/layout.php | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/views/layout.php b/views/layout.php
index da97687..a154a84 100644
--- a/views/layout.php
+++ b/views/layout.php
@@ -27,6 +27,10 @@
 
     <link rel="icon" href="/favicon.ico" type="image/x-icon">
 
+    <meta name="apple-mobile-web-app-capable" content="yes">
+    <meta name="mobile-web-app-capable" content="yes">
+    <meta name="theme-color" content="#428bca">
+
     <script src="/js/jquery-1.7.1.min.js"></script>
     <script src="/js/script.js"></script>
     <script src="/js/date.js"></script>
-- 
cgit v1.2.3


From 6ba8cf48a87c8aaa18ebc04ec65ad8c6e388a081 Mon Sep 17 00:00:00 2001
From: Prateek Saxena <prtksxna@gmail.com>
Date: Mon, 25 Apr 2016 18:34:57 +0530
Subject: Remove vendor prefix for border-radius value

The -moz one was an invalid property too.
---
 public/css/style.css | 20 +++++---------------
 1 file changed, 5 insertions(+), 15 deletions(-)

diff --git a/public/css/style.css b/public/css/style.css
index d806c62..f9f9ac2 100644
--- a/public/css/style.css
+++ b/public/css/style.css
@@ -13,7 +13,7 @@ body {
 .narrow {
   max-width: 700px;
   margin-right: auto;
-  margin-left: auto;  
+  margin-left: auto;
 }
 
 .footer {
@@ -28,7 +28,7 @@ body {
 .footer .nav {
   max-width: 700px;
   margin-right: auto;
-  margin-left: auto;  
+  margin-left: auto;
 }
 
 .footer .credits {
@@ -65,7 +65,7 @@ body {
 .bs-callout .highlight {
   background-color: #fff;
 }
- 
+
 /* Themes for different contexts */
 .bs-callout-danger {
   background-color: #fcf2f2;
@@ -132,7 +132,7 @@ body {
     float: left;
     margin-left: 10px;
   }
-  
+
   #last_response_date {
     font-size: 80%;
     font-weight: normal;
@@ -160,10 +160,6 @@ body {
   }
 
   .img-visible {
-    -webkit-border-bottom-right-radius: 0;
-    -webkit-border-bottom-left-radius: 0;
-    -moz-border-radius-bottomright: 0;
-    -moz-border-radius-bottomleft: 0;
     border-bottom-right-radius: 0;
     border-bottom-left-radius: 0;
   }
@@ -171,10 +167,6 @@ body {
   #note_location_img img {
     margin-top: -1px;
     border: 1px solid #ccc;
-    -webkit-border-bottom-right-radius: 4px;
-    -webkit-border-bottom-left-radius: 4px;
-    -moz-border-radius-bottomright: 4px;
-    -moz-border-radius-bottomleft: 4px;
     border-bottom-right-radius: 4px;
     border-bottom-left-radius: 4px;
   }
@@ -194,7 +186,7 @@ body {
       cursor: pointer;
       opacity: 0;
       filter: alpha(opacity=0);
-  }  
+  }
 
   .callout {
     border-left: 4px #5bc0de solid;
@@ -205,5 +197,3 @@ body {
   .callout table {
     margin-bottom: 0;
   }
-
-
-- 
cgit v1.2.3