From 77135214acb9c2207c789d89a892c8652d474a9f Mon Sep 17 00:00:00 2001 From: Sarah Norris Date: Tue, 2 Nov 2021 23:03:23 +0000 Subject: [PATCH 1/6] Add important to active text color --- videomaker/assets/theme.css | 2 +- videomaker/sass/_links.scss | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/videomaker/assets/theme.css b/videomaker/assets/theme.css index 1c5e4c830e..08d7f2d3bb 100644 --- a/videomaker/assets/theme.css +++ b/videomaker/assets/theme.css @@ -81,7 +81,7 @@ a:active { background: var(--wp--custom--color--foreground); - color: var(--wp--custom--color--background); + color: var(--wp--custom--color--background) !important; } div.post-meta.wp-block-group { diff --git a/videomaker/sass/_links.scss b/videomaker/sass/_links.scss index 07270c763f..82db53cdef 100644 --- a/videomaker/sass/_links.scss +++ b/videomaker/sass/_links.scss @@ -1,4 +1,4 @@ a:active { background: var(--wp--custom--color--foreground); - color: var(--wp--custom--color--background); + color: var(--wp--custom--color--background) !important; } From bef042eb5b9d96245bb01f9ed648c6ccdaa4f5c1 Mon Sep 17 00:00:00 2001 From: Sarah Norris Date: Tue, 2 Nov 2021 23:15:50 +0000 Subject: [PATCH 2/6] Add underline to nav items on hover --- videomaker/assets/theme.css | 4 ++++ videomaker/sass/_navigation.scss | 4 ++++ 2 files changed, 8 insertions(+) diff --git a/videomaker/assets/theme.css b/videomaker/assets/theme.css index 08d7f2d3bb..4fd32634db 100644 --- a/videomaker/assets/theme.css +++ b/videomaker/assets/theme.css @@ -111,6 +111,10 @@ div.post-meta a { row-gap: 0rem; } +.wp-block-navigation.is-style-blockbase-navigation-improved-responsive.is-responsive .is-menu-open.wp-block-navigation__responsive-container .wp-block-navigation__responsive-container-content .wp-block-navigation-item > a:hover { + text-decoration-line: underline; +} + .wp-block-navigation.is-style-blockbase-navigation-improved-responsive.is-responsive .is-menu-open.wp-block-navigation__responsive-container .wp-block-navigation__responsive-container-content .wp-block-navigation-item.current-menu-item > a { text-decoration: underline; } diff --git a/videomaker/sass/_navigation.scss b/videomaker/sass/_navigation.scss index 8fcbcddc86..a11a9fadb0 100644 --- a/videomaker/sass/_navigation.scss +++ b/videomaker/sass/_navigation.scss @@ -13,6 +13,10 @@ .wp-block-navigation-item { row-gap: 0rem; + > a:hover { + text-decoration-line: underline; + } + &.current-menu-item > a { text-decoration: underline; } From c56ed1cd0557b34eb921a66467d9984c80109c02 Mon Sep 17 00:00:00 2001 From: Sarah Norris Date: Tue, 2 Nov 2021 23:16:54 +0000 Subject: [PATCH 3/6] Set nav to always be a hamburger menu --- videomaker/block-template-parts/header.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/videomaker/block-template-parts/header.html b/videomaker/block-template-parts/header.html index 754a4ff65e..2a08c51dc5 100644 --- a/videomaker/block-template-parts/header.html +++ b/videomaker/block-template-parts/header.html @@ -3,6 +3,6 @@ - + From 8c529a088fa3c7da75d2b3a5c953b9979db9fd76 Mon Sep 17 00:00:00 2001 From: Sarah Norris Date: Tue, 2 Nov 2021 23:25:38 +0000 Subject: [PATCH 4/6] Left align open hamburger menu --- videomaker/assets/theme.css | 4 ++++ videomaker/sass/_navigation.scss | 2 ++ 2 files changed, 6 insertions(+) diff --git a/videomaker/assets/theme.css b/videomaker/assets/theme.css index 4fd32634db..38f6f7bc2b 100644 --- a/videomaker/assets/theme.css +++ b/videomaker/assets/theme.css @@ -107,6 +107,10 @@ div.post-meta a { row-gap: 1rem; } +.wp-block-navigation.is-style-blockbase-navigation-improved-responsive.is-responsive .is-menu-open.wp-block-navigation__responsive-container .wp-block-navigation__responsive-container-content { + align-items: flex-start; +} + .wp-block-navigation.is-style-blockbase-navigation-improved-responsive.is-responsive .is-menu-open.wp-block-navigation__responsive-container .wp-block-navigation__responsive-container-content .wp-block-navigation-item { row-gap: 0rem; } diff --git a/videomaker/sass/_navigation.scss b/videomaker/sass/_navigation.scss index a11a9fadb0..f7dddaa289 100644 --- a/videomaker/sass/_navigation.scss +++ b/videomaker/sass/_navigation.scss @@ -10,6 +10,8 @@ } .wp-block-navigation__responsive-container-content { // Needed for specificity to beat the navigation block CSS + align-items: flex-start; + .wp-block-navigation-item { row-gap: 0rem; From fa6b940438e3ec4a8458479567493c83f1bb97bd Mon Sep 17 00:00:00 2001 From: Sarah Norris Date: Tue, 2 Nov 2021 23:43:05 +0000 Subject: [PATCH 5/6] Revert hamburger menu change Not needed in this PR --- videomaker/block-template-parts/header.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/videomaker/block-template-parts/header.html b/videomaker/block-template-parts/header.html index 2a08c51dc5..754a4ff65e 100644 --- a/videomaker/block-template-parts/header.html +++ b/videomaker/block-template-parts/header.html @@ -3,6 +3,6 @@ - + From a00c08fa72a44522fc06d4b9405f910ea839be69 Mon Sep 17 00:00:00 2001 From: Ben Dwyer Date: Thu, 4 Nov 2021 09:43:20 +0000 Subject: [PATCH 6/6] Update videomaker/sass/_links.scss --- videomaker/sass/_links.scss | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/videomaker/sass/_links.scss b/videomaker/sass/_links.scss index 82db53cdef..58f05aba81 100644 --- a/videomaker/sass/_links.scss +++ b/videomaker/sass/_links.scss @@ -1,4 +1,4 @@ a:active { - background: var(--wp--custom--color--foreground); + background: var(--wp--custom--color--foreground) !important; color: var(--wp--custom--color--background) !important; }