Issue 1950 - Fixed: Filter tooltip not shown for long filters

chrome/content/ui/filters-filteractions.js

Index: chrome/content/ui/filters-filteractions.js
===================================================================
--- a/chrome/content/ui/filters-filteractions.js
+++ b/chrome/content/ui/filters-filteractions.js
@@ -339,10 +339,17 @@
       while (box.firstChild)
         box.removeChild(box.firstChild);
 
-      for (var i = 0; i < text.length; i += 80)
+      let lines = text.match(/.{1,80}/g);
+      if (lines.length > 6)

[Erik, 2016/01/29 04:29:31]

I think this should be  `if (lines.length > 7)` since the logic here would
generate a 7 item array to replace `lines`.

If `lines` is only length 7 then we might as well show all of it.

[Thomas Greiner, 2016/01/29 18:28:55]

Done. I agree that the maximum of 482 characters that's specified in the ticket
should've rather been 560.

       {
-        var description = document.createElement("description");
-        description.setAttribute("value", text.substr(i, 80));
+        // Text is too long to display in full so we cut out the middle part
+        lines = lines.slice(0,3).concat("\u2026", lines.slice(-3));
+      }
+
+      for (let line of lines)
+      {
+        let description = document.createElement("description");
+        description.setAttribute("value", line);
         box.appendChild(description);
       }
     }