Skip to content

Commit

Permalink
Improve grouping of the secondary toolbar button CSS rules
Browse files Browse the repository at this point in the history
Using the `:is(a)` selector we can move the previously separate rules
into the main `.toolbarButton` override rules.

Co-authored-by: Calixte Denizet <[email protected]>
  • Loading branch information
timvandermeij and calixteman committed Aug 12, 2024
1 parent 2a982e7 commit 68bda90
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions web/viewer.css
Original file line number Diff line number Diff line change
Expand Up @@ -1247,6 +1247,16 @@ dialog :link {
white-space: normal;
width: auto;

&:is(a) {
padding-top: 5px;
text-decoration: none;

&[href="#"] {
opacity: 0.5;
pointer-events: none;
}
}

&::before {
inset-inline-start: 12px;
opacity: var(--doorhanger-icon-opacity);
Expand All @@ -1264,16 +1274,6 @@ dialog :link {
}
}

a.toolbarButton {
padding-top: 5px;
text-decoration: none;

&[href="#"] {
opacity: 0.5;
pointer-events: none;
}
}

#secondaryToolbarButtonContainer {
margin-bottom: -4px;
max-height: calc(var(--viewer-container-height) - 40px);
Expand Down

0 comments on commit 68bda90

Please sign in to comment.