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(-)

(limited to 'views')

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(-)

(limited to 'views')

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

(limited to 'views')

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(+)

(limited to 'views')

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