Browse Source

Fix menu type size and button padding regression.

Jeff Ong 4 years ago
parent
commit
2d2aa2212d

+ 6 - 0
spearhead/assets/sass/_header.scss

@@ -70,6 +70,8 @@ $site-branding-height: 68px;
 
 		button#primary-close-menu {
 			color: var(--global--color-secondary);
+			margin: 0;
+			padding: var(--global--spacing-horizontal);
 			position: fixed;
 			top: 0;
 			right: 0;
@@ -145,6 +147,10 @@ $site-branding-height: 68px;
 
 .site-header div.menu-button-container {
 	button#primary-open-menu {
+		margin-top: 0;
+		margin-right: 0;
+		padding: var(--global--spacing-horizontal);
+
 		&:before {
 			margin: 0;
 		}

+ 8 - 0
spearhead/style-rtl.css

@@ -201,6 +201,8 @@ Tags: one-column, flexible-header, accessibility-ready, custom-colors, custom-me
 
 .site-header .primary-navigation button#primary-close-menu {
 	color: var(--global--color-secondary);
+	margin: 0;
+	padding: var(--global--spacing-horizontal);
 	position: fixed;
 	top: 0;
 	left: 0;
@@ -272,6 +274,12 @@ Tags: one-column, flexible-header, accessibility-ready, custom-colors, custom-me
 	color: var(--global--color-primary);
 }
 
+.site-header div.menu-button-container button#primary-open-menu {
+	margin-top: 0;
+	margin-left: 0;
+	padding: var(--global--spacing-horizontal);
+}
+
 .site-header div.menu-button-container button#primary-open-menu:before {
 	margin: 0;
 }

+ 8 - 0
spearhead/style.css

@@ -201,6 +201,8 @@ Tags: one-column, flexible-header, accessibility-ready, custom-colors, custom-me
 
 .site-header .primary-navigation button#primary-close-menu {
 	color: var(--global--color-secondary);
+	margin: 0;
+	padding: var(--global--spacing-horizontal);
 	position: fixed;
 	top: 0;
 	right: 0;
@@ -272,6 +274,12 @@ Tags: one-column, flexible-header, accessibility-ready, custom-colors, custom-me
 	color: var(--global--color-primary);
 }
 
+.site-header div.menu-button-container button#primary-open-menu {
+	margin-top: 0;
+	margin-right: 0;
+	padding: var(--global--spacing-horizontal);
+}
+
 .site-header div.menu-button-container button#primary-open-menu:before {
 	margin: 0;
 }

File diff suppressed because it is too large
+ 0 - 0
spearhead/style.css.map


+ 3 - 0
spearhead/variables.css

@@ -68,6 +68,7 @@
 	--primary-nav--font-family-mobile: var(--global--font-family-secondary);
 	--primary-nav--font-size-mobile: var(--global--font-size-sm);
 	--primary-nav--font-size-sub-menu-mobile: var( --primary-nav--font-size-mobile );
+	--primary-nav--font-size: var(--global--font-size-sm);
 
 	--social-nav--padding: var(--global--spacing-unit);
 
@@ -103,5 +104,7 @@
 		--global--color-text-selection: #000000;
 		--primary-nav--color-background: var(--global--color-foreground-light);
 		--sticky-posts--color-background: #1E1F21;
+		--button--padding-vertical: 24px;
+		--button--padding-horizontal: 32px;
 	}
 }

Some files were not shown because too many files changed in this diff