From 2828e8bba36a2c6df0d447c64c25aa3974675474 Mon Sep 17 00:00:00 2001 From: narcolepticinsomniac Date: Sat, 2 Dec 2017 18:42:57 -0500 Subject: [PATCH] Subtlety >an animated checkbox needlessly draws attention to itself But... I feel the need. Drawing attention to the checkbox is intentional to indicate click action. >In case the entire row is highlighted there will be no distraction Clickable area isn't the entire row, it's everything left of action icons. Hovering background on part of the row will look odd, and it's also a less obvious indicator of click action IMO. Didn't change "disable all" yet. It's a special kinda function, so maybe slightly more drastic is warranted, but I can tone that down too. --- popup/popup.css | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/popup/popup.css b/popup/popup.css index 43adf4c8..66720d9d 100644 --- a/popup/popup.css +++ b/popup/popup.css @@ -32,11 +32,6 @@ body > div:not(#installed) { margin: 2px 0 0 2px; } -input[type="checkbox"]:checked:hover + .svg-icon.checked, -.style-name:hover input[type="checkbox"]:checked + .svg-icon.checked { - fill: #fff; -} - input[type="checkbox"]:checked + .svg-icon.checked { display: inline-flex; transition: fill .25s; @@ -57,12 +52,14 @@ input[type="checkbox"] { } input[type="checkbox"]:checked:hover, .style-name:hover input[type="checkbox"]:checked { - background-color: hsl(0, 0%, 46%); + border-color: hsl(0, 0%, 32%); + background-color: hsl(0, 0%, 82%); } input[type="checkbox"]:hover, .style-name:hover input[type="checkbox"], .entry.disabled .style-name:hover input[type="checkbox"] { - background-color: hsl(0, 0%, 76%); + border-color: hsl(0, 0%, 32%); + background-color: hsl(0, 0%, 82%); } #installed.disabled + .actions #disableAll:checked + .svg-icon.checked {