Move overlay-header styles to Overlay

.header > form matches no element on the page
all other selectors are linked to the mobile Overlay so should
be shipped along with those styles.

Remove .header-action rule, given that header-action's only appear
inside Overlays (Editor, Category and Talk)

A more generic rule already exists (.overlay .header-action > *)
for overlays that sets the min-height

Change-Id: I114fe80e50bb833e9e6bb0fb6b13c2e929244ddf
This commit is contained in:
jdlrobson 2018-07-05 16:03:04 -07:00 committed by Jdlrobson
parent ef63ced9b4
commit 8351c88c50
2 changed files with 12 additions and 17 deletions

View File

@ -108,23 +108,6 @@ body {
> .header-title {
vertical-align: middle;
}
.header-action > * {
min-height: @headerHeight;
}
}
.header > form,
.overlay-header .overlay-title {
padding: 0.15em 0;
&:last-child {
// Reserve space to the right in case the search form in the header or
// overlay title in the overlay header is the last element. Use @iconGutterWidth
// because there is a hamburger or close/back icon next to this element and
// the space between them is exactly that.
padding-right: @iconGutterWidth;
}
}
#searchInput {

View File

@ -24,6 +24,18 @@
}
}
.overlay-header .overlay-title {
padding: 0.15em 0;
&:last-child {
// Reserve space to the right in case the search form in the header or
// overlay title in the overlay header is the last element. Use @iconGutterWidth
// because there is a hamburger or close/back icon next to this element and
// the space between them is exactly that.
padding-right: @iconGutterWidth;
}
}
@media all and ( min-width: @width-breakpoint-desktop ) {
.overlay-header {
// Make sure the close button and secondary button icon images are aligned