Skip to content

Commit

Permalink
Slim down the height of the announcement bar on desktop (#2807)
Browse files Browse the repository at this point in the history
* Adjust height of announcement bar and sub-items.

* Adjust announcement bar button height on desktop only.

* Fix width of the utility bar on mobile.

* Remove bizarre rem-based values.
  • Loading branch information
kjellr committed Jul 10, 2023
1 parent f10b784 commit 1d70bdc
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 2 deletions.
13 changes: 12 additions & 1 deletion assets/base.css
Original file line number Diff line number Diff line change
Expand Up @@ -112,12 +112,18 @@ body:has(.section-header .drawer-menu) .announcement-bar-section .page-width {
margin: 0 auto;
}

.utility-bar__grid.page-width {
padding-left: 3rem;
padding-right: 3rem;
}

@media screen and (min-width: 750px) {
.page-width {
padding: 0 5rem;
}

.header.page-width {
.header.page-width,
.utility-bar__grid.page-width {
padding-left: 3.2rem;
padding-right: 3.2rem;
}
Expand Down Expand Up @@ -2233,6 +2239,10 @@ product-info .loading-overlay:not(.hidden) ~ *,
.announcement-bar-slider {
width: 60%;
}

.announcement-bar .slider-button {
height: 3.8rem;
}
}

.announcement-bar__link {
Expand Down Expand Up @@ -2266,6 +2276,7 @@ product-info .loading-overlay:not(.hidden) ~ *,
padding: 1rem 0;
margin: 0;
letter-spacing: 0.1rem;
min-height: 3.8rem;
}

.announcement-bar-slider--fade-in-next .announcement-bar__message,
Expand Down
5 changes: 5 additions & 0 deletions assets/component-list-social.css
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,11 @@
color: rgb(var(--color-foreground));
}

.utility-bar .list-social__link {
padding: 0 1.1rem;
height: 3.8rem;
}

.list-social__link:hover .icon {
transform: scale(1.07);
}
4 changes: 3 additions & 1 deletion assets/component-localization-form.css
Original file line number Diff line number Diff line change
Expand Up @@ -193,12 +193,14 @@ noscript .localization-selector.link {
/* Header localization */
.header-localization .localization-form:only-child {
margin: 0;
padding: 0;
}

.header-localization .disclosure .localization-form__select {
font-size: 1.4rem;
letter-spacing: 0.06rem;
height: auto;
min-height: initial;
background: transparent;
}

Expand Down Expand Up @@ -234,12 +236,12 @@ noscript .localization-selector.link {

.header-localization:not(.menu-drawer__localization) {
align-items: center;
min-height: 4.4rem;
}

.js .header-localization:not(.menu-drawer__localization) .localization-form__select {
padding: 0 2.7rem 0 1.2rem;
width: max-content;
height: 3.8rem;
}

.header-localization:not(.menu-drawer__localization) .localization-form:only-child .localization-form__select {
Expand Down

0 comments on commit 1d70bdc

Please sign in to comment.