Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use background-color rather than background in a few CSS rules

Browse Source 
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.
This commit is contained in:
Jonas Jenwald 2022-04-12 12:33:51 +02:00
parent 3aed103ce8
commit 93c5c6e5e9
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
web/viewer.css
View File

@ -1254,7 +1254,7 @@ a:focus > .thumbnail > .thumbnailSelectionRing,
} }
#errorWrapper { #errorWrapper {
background: none repeat scroll 0 0 var(--errorWrapper-bg-color); background-color: var(--errorWrapper-bg-color);
color: var(--main-color); color: var(--main-color);
left: 0; left: 0;
position: absolute; position: absolute;
@ -1364,7 +1364,7 @@ dialog :link {
} }
#PDFBug { #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); border: 1px solid rgba(102, 102, 102, 1);
position: fixed; position: fixed;
top: 32px; top: 32px;
@ -1393,12 +1393,9 @@ dialog :link {
#PDFBug button.active { #PDFBug button.active {
font-weight: bold; font-weight: bold;
} }
.debuggerShowText { .debuggerShowText,
background: none repeat scroll 0 0 rgba(255, 255, 0, 1);
color: rgba(0, 0, 255, 1);
}
.debuggerHideText:hover { .debuggerHideText:hover {
background: none repeat scroll 0 0 rgba(255, 255, 0, 1); background-color: rgba(255, 255, 0, 1);
} }
#PDFBug .stats { #PDFBug .stats {
font-family: courier; font-family: courier;