From 8e5de4199aded1b9af2201daef85d6f24cfa5679 Mon Sep 17 00:00:00 2001
From: amatriain <geralt@gmail.com>
Date: Tue, 26 Jan 2016 11:57:31 +0100
Subject: [PATCH] Fix displaying the number of unread entries in the favicon
 for Chrome.

Currently favico.js has a bug which only updates the last element with rel="icon" if there are several in the page head. This maeans that the favicon must be last in the head, particularly it must appear after the 192x192 icon (used in the mobile manifest).

Related to this favico.js bug, might become unnecessary when it is fixed:

  https://github.com/ejci/favico.js/issues/85
---
 app/views/layouts/_header.html.erb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/views/layouts/_header.html.erb b/app/views/layouts/_header.html.erb
index 93c1adc00..5c7e24de0 100644
--- a/app/views/layouts/_header.html.erb
+++ b/app/views/layouts/_header.html.erb
@@ -6,7 +6,6 @@
     <meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
     <meta name="mobile-web-app-capable" content="yes">
     <meta name="apple-mobile-web-app-capable" content="yes">
-    <%= favicon_link_tag '/favicon.ico' %>
 <% end %>
     <%= stylesheet_link_tag    'application', :media => 'all' %>
 
@@ -14,4 +13,6 @@
     <link rel="manifest" href="<%= page_path 'manifest.json' %>">
     <meta name="theme-color" content="#FFA500">
     <%= csrf_meta_tags %>
+    <%# favicon must be the last element with rel="icon", otherwise favico.js won't work %>
+    <%= favicon_link_tag '/favicon.ico' %>
   </head>