darrinhenein / VerticalTabs

Firefox add-on for arranging tabs vertically
https://addons.mozilla.org/firefox/addon/108862/
194 stars 28 forks source link

Linux: Too high CPU load #8

Open noscript opened 9 years ago

noscript commented 9 years ago

Using FF 31.0

noscript commented 9 years ago

The culprit is in the skin stylesheet. I have no idea why, but having the tab height more than 21px causes constant ~50% CPU load. There is my fix:

--- a/skin/linux/linux.css  2014-09-01 19:16:43.300756000 +0300
+++ b/skin/linux/linux.css  2014-09-01 19:17:00.958383000 +0300
@@ -19,8 +19,8 @@
     margin: 0 !important;
     padding: 1px 0 !important;
     height: 24px !important;
-    min-height: 24px !important;
-    max-height: 24px !important;
+    min-height: 21px !important;
+    max-height: 21px !important;
 }

 .tabbrowser-tab[selected="true"],
AWilyFox commented 9 years ago

Confirmed that this workaround works in FF35. It also re-enables scrolling in the vertical tabs list, which is issue #10