- Status changed to Needs review
12 months ago 10:52am 22 November 2023
According to bartik.breakpoints.yml the breakpoint where mobile goes into narrow is 560px:
bartik.mobile:
label: mobile
mediaQuery: ''
weight: 0
multipliers:
- 1x
bartik.narrow:
label: narrow
mediaQuery: 'all and (min-width: 560px) and (max-width: 850px)'
weight: 1
multipliers:
- 1x
bartik.wide:
label: wide
mediaQuery: 'all and (min-width: 851px)'
weight: 2
multipliers:
- 1x
Yet, the CSS media queries for the "hamburger menu" state 460px:
diff --git a/docroot/core/themes/bartik/css/components/header.css b/docroot/core/themes/bartik/css/components/header.css
index 40ea0f0..9ec9ce2 100644
--- a/docroot/core/themes/bartik/css/components/header.css
+++ b/docroot/core/themes/bartik/css/components/header.css
@@ -11,7 +11,7 @@
.region-header .site-branding {
margin-top: 0.429em;
}
-@media all and (min-width: 461px) {
+@media all and (min-width: 561px) {
.region-header .block {
float: right; /* LTR */
margin-top: 0.357em;
diff --git a/docroot/core/themes/bartik/css/components/primary-menu.css b/docroot/core/themes/bartik/css/components/primary-menu.css
index 1132ae2..7aa15e3 100644
--- a/docroot/core/themes/bartik/css/components/primary-menu.css
+++ b/docroot/core/themes/bartik/css/components/primary-menu.css
@@ -111,7 +111,7 @@ body:not(:target) .region-primary-menu .menu-toggle-target-show:target ~ .menu .
/**
* Media queries for primary menu.
*/
-@media all and (min-width: 461px) and (max-width: 900px) {
+@media all and (min-width: 561px) and (max-width: 900px) {
.region-primary-menu .menu {
margin: 0 5px;
padding: 0;
@@ -203,9 +203,9 @@ body:not(:target) .region-primary-menu .menu-toggle-target-show:target ~ .menu .
/**
* Ensures that the open mobile menu hides when the screen dimensions become
- * 461px or wider.
+ * 561px or wider.
*/
-@media all and (min-width: 461px) {
+@media all and (min-width: 561px) {
body:not(:target) .region-primary-menu .menu-toggle-target-show:target ~ .menu-toggle--hide {
display: none;
}
diff --git a/docroot/core/themes/bartik/css/components/site-branding.css b/docroot/core/themes/bartik/css/components/site-branding.css
index fda5d9e..cc947a8 100644
--- a/docroot/core/themes/bartik/css/components/site-branding.css
+++ b/docroot/core/themes/bartik/css/components/site-branding.css
@@ -16,7 +16,7 @@
display: inline-block;
vertical-align: top;
}
-@media all and (min-width: 461px) {
+@media all and (min-width: 561px) {
.site-branding__text {
margin-bottom: 1.857em;
}
Needs review
1.0
Look and Feel
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.