Issue 6510 - Configure notification opens options page incorrectly

css/desktop-options.scss

Index: css/desktop-options.scss
===================================================================
--- a/css/desktop-options.scss
+++ b/css/desktop-options.scss
@@ -533,16 +533,33 @@
 }
 
 [data-validation] .floating-input input:focus:invalid ~ .error-msg
 {
   visibility: visible;
 }
 
 /*
+  Animations
+*/
+
+.highlight-animate
+{
+  animation: highlight 1s 3;
+}
+
+@keyframes highlight
+{
+  0% { background: none }

[a.giammarchi, 2018/03/22 18:43:28]

I think the neutral value for background-color is transparent.

`none` IIRC is only for background-image, to nullify any data/url.

[saroyanm, 2018/03/22 20:43:21]

Done.

+  30% { background: #ffd7a3 }

[a.giammarchi, 2018/03/22 18:43:28]

whenever you're changing only one part of a complex CSS property, is usually
better to address it directly. background is a mess of overloads and it's able
to overwrite many other things if set directly. background-color though, is what
you are after here, so I'd go with background-color.

[saroyanm, 2018/03/22 20:43:21]

Agree, done.

+  70% { background: #ffd7a3 }
+  100% { background: none }
+}
+
+/*
   Sidebar
  */
 
 #sidebar,
 #sidebar .fixed,
 [role="tablist"]
 {
   width: 14.3rem;