Bug 1728368 - Simplify findbar overflow behavior. r=emalysz

Differential Revision: https://phabricator.services.mozilla.com/D124086
This commit is contained in:
Dão Gottwald 2021-09-01 02:43:24 +00:00
parent c872d06387
commit f97f02f5a8
3 changed files with 3 additions and 11 deletions

View File

@ -280,14 +280,12 @@ menupopup::part(drop-indicator) {
/* Find bar */
.browserContainer > findbar > #findbar-close-container,
.browserContainer > findbar {
background-color: var(--toolbar-non-lwt-bgcolor);
color: var(--toolbar-non-lwt-textcolor);
text-shadow: none;
}
:root:not([lwtheme-image]) .browserContainer > findbar:-moz-lwtheme > #findbar-close-container,
:root:not([lwtheme-image]) .browserContainer > findbar:-moz-lwtheme {
background-color: var(--lwt-accent-color);
background-image: linear-gradient(var(--toolbar-bgcolor), var(--toolbar-bgcolor));
@ -295,7 +293,6 @@ menupopup::part(drop-indicator) {
color: var(--toolbar-color);
}
:root:not([lwtheme-image]) .browserContainer > findbar:-moz-lwtheme:-moz-window-inactive > #findbar-close-container,
:root:not([lwtheme-image]) .browserContainer > findbar:-moz-lwtheme:-moz-window-inactive {
background-color: var(--lwt-accent-color-inactive, var(--lwt-accent-color));
}

View File

@ -65,10 +65,8 @@
<image anonid="find-status-icon" class="find-status-icon" />
<description anonid="find-status" control="findbar-textbox" class="findbar-label findbar-find-status" />
</hbox>
<div id="findbar-close-container">
<toolbarbutton anonid="find-closebutton" class="findbar-closebutton tabbable close-icon"
data-l10n-id="findbar-find-button-close" oncommand="close();" />
</div>
<toolbarbutton anonid="find-closebutton" class="findbar-closebutton tabbable close-icon"
data-l10n-id="findbar-find-button-close" oncommand="close();"/>
`;
}

View File

@ -31,7 +31,7 @@ findbar[noanim] {
.findbar-container {
margin-inline-start: 8px;
height: 28px;
min-width: calc(100vw - 48px);
overflow: hidden; /* Ensures the close button stays visible. */
}
/* Remove start margin when close button is on the left side (on macOS) */
@ -189,9 +189,6 @@ html|input.findbar-textbox::placeholder,
}
/* Close button */
#findbar-close-container {
position: relative;
}
/* Increase specificity to override close-icon.css */
.close-icon.findbar-closebutton {