From 93c5c6e5e926fdd249d1b5d1ca386eba0e180785 Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Tue, 12 Apr 2022 12:33:51 +0200
Subject: [PATCH] Use `background-color` rather than `background` in a few CSS
 rules

In a couple of spots, mostly related to the debugging tools, we're unnecessarily using a somewhat "complex" `background`-format only to specify a solid color. This can be simplified by using `background-color` instead, and the patch also removes a `color`-rule that's being ignored anyway.
---
 web/viewer.css | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/web/viewer.css b/web/viewer.css
index b00ef85b1..1b0be67c8 100644
--- a/web/viewer.css
+++ b/web/viewer.css
@@ -1254,7 +1254,7 @@ a:focus > .thumbnail > .thumbnailSelectionRing,
 }
 
 #errorWrapper {
-  background: none repeat scroll 0 0 var(--errorWrapper-bg-color);
+  background-color: var(--errorWrapper-bg-color);
   color: var(--main-color);
   left: 0;
   position: absolute;
@@ -1364,7 +1364,7 @@ dialog :link {
 }
 
 #PDFBug {
-  background: none repeat scroll 0 0 rgba(255, 255, 255, 1);
+  background-color: rgba(255, 255, 255, 1);
   border: 1px solid rgba(102, 102, 102, 1);
   position: fixed;
   top: 32px;
@@ -1393,12 +1393,9 @@ dialog :link {
 #PDFBug button.active {
   font-weight: bold;
 }
-.debuggerShowText {
-  background: none repeat scroll 0 0 rgba(255, 255, 0, 1);
-  color: rgba(0, 0, 255, 1);
-}
+.debuggerShowText,
 .debuggerHideText:hover {
-  background: none repeat scroll 0 0 rgba(255, 255, 0, 1);
+  background-color: rgba(255, 255, 0, 1);
 }
 #PDFBug .stats {
   font-family: courier;