From 9dac68410d768e168361f0d10ff99ce3574da4a7 Mon Sep 17 00:00:00 2001 From: narcolepticinsomniac Date: Mon, 20 Mar 2017 05:58:38 -0400 Subject: [PATCH] Handle popup overflow better Max height is 600px for the popup, regardless of resolution (so says Google), so we can set the styles section to a max height in px and let the overflow scroll inside the div, instead of scrolling the whole popup. This way the other actions are always present at the bottom of the popup without needing to scroll to use them. I left a little breathing room for the rare instance when domain name in "write style for" is long enough to cause a line break, so there should never really be any overflow for the popup itself. https://github.com/schomery/stylish-chrome/pull/50#issuecomment-287665085 --- popup.css | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/popup.css b/popup.css index 60453d0b..e9c39907 100644 --- a/popup.css +++ b/popup.css @@ -58,7 +58,9 @@ body.blocked > DIV { border-bottom: none; } #installed { - margin-top: 0.5em; + padding-top: 2px; + max-height: 434px; + overflow-y: auto; } #installed.disabled .style-name { text-decoration: line-through;