From 04749a4aba34fceab587f4461a3c09a4058a8402 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Guillot?= <contact@fredericguillot.com>
Date: Wed, 10 Dec 2014 21:12:42 -0500
Subject: [PATCH] Replace mark as read link by an icon

---
 assets/css/app.css      |  7 ++++++-
 controllers/item.php    |  1 +
 templates/show_item.php | 14 ++++++++------
 3 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/assets/css/app.css b/assets/css/app.css
index 2878f77..d5e873f 100644
--- a/assets/css/app.css
+++ b/assets/css/app.css
@@ -638,11 +638,16 @@ span.downloading img {
     text-align: center;
 }
 
-a.bookmark-icon {
+a.icon {
     color: #333;
     text-decoration: none;
 }
 
+a.icon:focus,
+a.icon:hover {
+    color: #000;
+}
+
 #item-content-enclosure {
     border-bottom: 1px dashed #ccc;
     margin-bottom: 20px;
diff --git a/controllers/item.php b/controllers/item.php
index 6e363f0..1ff286a 100644
--- a/controllers/item.php
+++ b/controllers/item.php
@@ -46,6 +46,7 @@ Router\get_action('show', function() {
     $feed = Model\Feed\get($item['feed_id']);
 
     Model\Item\set_read($id);
+    $item['status'] = 'read';
 
     switch ($menu) {
         case 'unread':
diff --git a/templates/show_item.php b/templates/show_item.php
index dc8a0e0..295fa51 100644
--- a/templates/show_item.php
+++ b/templates/show_item.php
@@ -37,13 +37,20 @@
         <ul class="item-infos">
             <li>
                 <a
-                    class="bookmark-icon"
+                    class="bookmark-icon icon"
                     href="?action=bookmark&amp;value=<?= (int)!$item['bookmark'] ?>&amp;id=<?= $item['id'] ?>&amp;source=show&amp;menu=<?= $menu ?>"
                     title="<?= ($item['bookmark']) ? t('remove bookmark') : t('bookmark') ?>"
                     data-reverse-title="<?= ($item['bookmark']) ? t('bookmark') :t('remove bookmark') ?>"
                     data-action="bookmark"
                 ></a>
             </li>
+            <li>
+                <a
+                    href="?action=mark-item-unread&amp;id=<?= $item['id'] ?>&amp;redirect=<?= $menu ?>&amp;feed_id=<?= $item['feed_id'] ?>"
+                    title="<?= t('mark as unread') ?>"
+                    class="read-icon icon"
+                ></a>
+            </li>
             <li>
                 <a href="?action=feed-items&amp;feed_id=<?= $feed['id'] ?>"><?= Helper\escape($feed['title']) ?></a>
             </li>
@@ -55,11 +62,6 @@
                 <a href="<?= $item['enclosure'] ?>" rel="noreferrer" target="_blank"><?= t('attachment') ?></a>
             </li>
             <?php endif ?>
-            <li>
-                <a
-                    href="?action=mark-item-unread&amp;id=<?= $item['id'] ?>&amp;redirect=unread"
-                ><?= t('mark as unread') ?></a>
-            </li>
             <li class="hide-mobile">
                 <span id="download-item"
                       data-failure-message="<?= t('unable to fetch content') ?>"