From 536379434d8844bd27dedab1b316715de9e54be1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=81ukasz=20Mierzwa?= <l.mierzwa@gmail.com>
Date: Mon, 14 Aug 2017 13:02:08 -0700
Subject: [PATCH 1/3] Enable tooltips for history dropdown

It's a dropdown so data-toggle is already used for the menu
---
 assets/static/unsee.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/assets/static/unsee.js b/assets/static/unsee.js
index 49b5e1b9c..340800f75 100644
--- a/assets/static/unsee.js
+++ b/assets/static/unsee.js
@@ -338,7 +338,7 @@ function onReady(localStore) {
         $(".toggle").bootstrapSwitch();
 
         // enable tooltips, #settings is a dropdown so it already uses different data-toggle
-        $("[data-toggle='tooltip'], #settings").tooltip({
+        $("[data-toggle='tooltip'], #settings, #history").tooltip({
             trigger: "hover"
         });
 

From 15b3072a336d85a0f00327b289d0e84303c72f6c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=81ukasz=20Mierzwa?= <l.mierzwa@gmail.com>
Date: Mon, 14 Aug 2017 15:54:50 -0700
Subject: [PATCH 2/3] Render search history and default/saved filter separator
 only if needed

If there are no recorded search filters than we don't need any separator
---
 assets/templates/history.html | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/assets/templates/history.html b/assets/templates/history.html
index 55c8e9943..d4f95e0fd 100644
--- a/assets/templates/history.html
+++ b/assets/templates/history.html
@@ -1,13 +1,17 @@
 <script type="application/json" id="history-menu">
+<% let filterRendered = false %>
 <% if (filters.length) { %>
   <% _.each(filters, function(filter) { %>
     <% if (filter !== activeFilter) { %>
       <%= renderTemplate("historyMenuItem", {filter: filter, icon: "fa fa-search"}) %>
+      <% filterRendered = true %>
     <% } %>
   <% }) %>
 <% } %>
 <% if (defaultFilter || savedFilter) { %>
-  <li role="separator" class="divider"></li>
+  <% if (filterRendered) { %>
+    <li role="separator" class="divider"></li>
+  <% } %>
   <% if (defaultFilter) { %>
     <%= renderTemplate("historyMenuItem", {filter: defaultFilter, icon: "fa fa-home"}) %>
   <% } %>

From c6e91c658e197855b0f7cc680414838271f73176 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=81ukasz=20Mierzwa?= <l.mierzwa@gmail.com>
Date: Mon, 14 Aug 2017 16:04:02 -0700
Subject: [PATCH 3/3] Update test snapshots

Rendered HTML changed, so tests need updating
---
 .../static/__snapshots__/filters.test.js.snap | 42 +++++++++++++------
 1 file changed, 29 insertions(+), 13 deletions(-)

diff --git a/assets/static/__snapshots__/filters.test.js.snap b/assets/static/__snapshots__/filters.test.js.snap
index 91b106da6..bdd733d54 100644
--- a/assets/static/__snapshots__/filters.test.js.snap
+++ b/assets/static/__snapshots__/filters.test.js.snap
@@ -1,10 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`appended filtes should be present in history 1`] = `
-"<li role=\\"separator\\" class=\\"divider\\"></li>
-  
-    
-<li class=\\"history-menu\\">
+"<li class=\\"history-menu\\">
   <a class=\\"cursor-pointer history-menu-item\\">
     <span class=\\"rawFilter hidden\\">
       default
@@ -34,11 +31,14 @@ exports[`appended filtes should be present in history 2`] = `
   </a>
 </li>
 
+      
     
   
 
 
-  <li role=\\"separator\\" class=\\"divider\\"></li>
+  
+    <li role=\\"separator\\" class=\\"divider\\"></li>
+  
   
     
 <li class=\\"history-menu\\">
@@ -75,6 +75,7 @@ exports[`appended filtes should be present in history 3`] = `
   </a>
 </li>
 
+      
     
   
     
@@ -93,11 +94,14 @@ exports[`appended filtes should be present in history 3`] = `
   </a>
 </li>
 
+      
     
   
 
 
-  <li role=\\"separator\\" class=\\"divider\\"></li>
+  
+    <li role=\\"separator\\" class=\\"divider\\"></li>
+  
   
     
 <li class=\\"history-menu\\">
@@ -138,6 +142,7 @@ exports[`appended filtes should be present in history 4`] = `
   </a>
 </li>
 
+      
     
   
     
@@ -160,6 +165,7 @@ exports[`appended filtes should be present in history 4`] = `
   </a>
 </li>
 
+      
     
   
     
@@ -178,11 +184,14 @@ exports[`appended filtes should be present in history 4`] = `
   </a>
 </li>
 
+      
     
   
 
 
-  <li role=\\"separator\\" class=\\"divider\\"></li>
+  
+    <li role=\\"separator\\" class=\\"divider\\"></li>
+  
   
     
 <li class=\\"history-menu\\">
@@ -223,6 +232,7 @@ exports[`appended filtes should be present in history 5`] = `
   </a>
 </li>
 
+      
     
   
     
@@ -245,6 +255,7 @@ exports[`appended filtes should be present in history 5`] = `
   </a>
 </li>
 
+      
     
   
     
@@ -263,11 +274,14 @@ exports[`appended filtes should be present in history 5`] = `
   </a>
 </li>
 
+      
     
   
 
 
-  <li role=\\"separator\\" class=\\"divider\\"></li>
+  
+    <li role=\\"separator\\" class=\\"divider\\"></li>
+  
   
     
 <li class=\\"history-menu\\">
@@ -300,6 +314,7 @@ exports[`appended filtes should be present in history 6`] = `
   </a>
 </li>
 
+      
     
   
     
@@ -326,6 +341,7 @@ exports[`appended filtes should be present in history 6`] = `
   </a>
 </li>
 
+      
     
   
     
@@ -348,11 +364,14 @@ exports[`appended filtes should be present in history 6`] = `
   </a>
 </li>
 
+      
     
   
 
 
-  <li role=\\"separator\\" class=\\"divider\\"></li>
+  
+    <li role=\\"separator\\" class=\\"divider\\"></li>
+  
   
     
 <li class=\\"history-menu\\">
@@ -371,10 +390,7 @@ exports[`appended filtes should be present in history 6`] = `
 `;
 
 exports[`default filter should be in history after setting filter to foo 1`] = `
-"<li role=\\"separator\\" class=\\"divider\\"></li>
-  
-    
-<li class=\\"history-menu\\">
+"<li class=\\"history-menu\\">
   <a class=\\"cursor-pointer history-menu-item\\">
     <span class=\\"rawFilter hidden\\">
       default