From 42340583582c11d4acc3008f36ad91cbfec279c6 Mon Sep 17 00:00:00 2001
From: Joe <33972521+hnb-ku@users.noreply.github.com>
Date: Fri, 2 Nov 2018 06:18:07 +0800
Subject: [PATCH] UX: don't show crawler navigation in print view (#6551)

* UX: adds CSS classes to crawler navigation links

* UX: hide crawler navigation in print view
---
 app/views/layouts/crawler.html.erb | 4 ++--
 app/views/topics/show.html.erb     | 8 +++++++-
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/app/views/layouts/crawler.html.erb b/app/views/layouts/crawler.html.erb
index 85495538c0f..8a485633cc9 100644
--- a/app/views/layouts/crawler.html.erb
+++ b/app/views/layouts/crawler.html.erb
@@ -28,14 +28,14 @@
       <%= yield %>
     </div>
     <footer class="container">
-      <nav itemscope itemtype='http://schema.org/SiteNavigationElement'>
+      <nav class='crawler-nav' itemscope itemtype='http://schema.org/SiteNavigationElement'>
         <a href='<%= path "/" %>'><%= t 'home_title' %></a>
         <%= link_to t('js.filters.categories.title'), path("/categories") %>
         <%= link_to t('guidelines_topic.title'), path("/guidelines") %>
         <%= link_to t('tos_topic.title'), path("/tos") %>
         <%= link_to t('privacy_topic.title'), path("/privacy") %>
       </nav>
-      <p><%= t 'powered_by_html' %></p>
+      <p class='powered-by-link'><%= t 'powered_by_html' %></p>
     </footer>
     <%= theme_lookup("body_tag") %>
   </body>
diff --git a/app/views/topics/show.html.erb b/app/views/topics/show.html.erb
index f0d02c9d8db..15b72d539f1 100644
--- a/app/views/topics/show.html.erb
+++ b/app/views/topics/show.html.erb
@@ -106,9 +106,15 @@
   <% content_for :after_body do %>
       <style media="print">
         /* For readability */
-        a, a:visited {
+        a,
+        a:visited {
           color: #0088cc !important;
         }
+        /* hide crawler nav in print view */
+        .crawler-nav,
+        .powered-by-link {
+          display: none;
+        }
       </style>
       <%= preload_script('print-page') %>
   <% end %>