Skip to content

Commit

Permalink
Merge pull request #18597 from timvandermeij/css-secondary-toolbar-pa…
Browse files Browse the repository at this point in the history
…rt-3

Group and scope the secondary toolbar rules using CSS nesting
  • Loading branch information
timvandermeij authored Aug 12, 2024
2 parents aa2337f + 5193adf commit 9103cf0
Showing 1 changed file with 20 additions and 5 deletions.
25 changes: 20 additions & 5 deletions web/viewer.css
Original file line number Diff line number Diff line change
Expand Up @@ -334,8 +334,7 @@ body {
}

:is(.toolbar, .editorParamsToolbar, #sidebarContainer)
:is(input, button, select),
.secondaryToolbar :is(input, button, a, select) {
:is(input, button, select) {
outline: none;
font: message-box;
}
Expand All @@ -362,7 +361,6 @@ body {
}

#toolbarContainer,
.secondaryToolbar,
.editorParamsToolbar {
position: relative;
height: var(--toolbar-height);
Expand Down Expand Up @@ -447,7 +445,6 @@ body {
transition-duration: 0s;
}

.secondaryToolbar,
.editorParamsToolbar {
top: var(--toolbar-height);
position: absolute;
Expand All @@ -462,7 +459,6 @@ body {
cursor: default;
}

.secondaryToolbar,
.editorParamsToolbar {
padding: 6px 0 10px;
inset-inline-end: 4px;
Expand Down Expand Up @@ -1223,6 +1219,25 @@ dialog :link {
}

#secondaryToolbar {
background-color: var(--doorhanger-bg-color);
cursor: default;
font: message-box;
font-size: 12px;
height: auto;
inset-inline-end: 4px;
line-height: 14px;
margin: 4px 2px;
padding: 6px 0 10px;
position: absolute;
text-align: left;
top: var(--toolbar-height);
z-index: 30000;

:is(button, a) {
font: message-box;
outline: none;
}

.toolbarButton {
border-radius: 0;
display: inline-block;
Expand Down

0 comments on commit 9103cf0

Please sign in to comment.