Skip to content

Commit cf4d146

Browse files
authored
Theme Showcase: Post design review polish (modern version) (#109388)
1 parent ae39efe commit cf4d146

File tree

10 files changed

+60
-16
lines changed

10 files changed

+60
-16
lines changed

client/components/theme-tier/theme-tier-badge/style.scss

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,4 +85,11 @@
8585
.components-button {
8686
font-size: inherit;
8787
}
88-
}
88+
}
89+
90+
.is-theme-showcase-modern {
91+
.premium-badge,
92+
.theme-tier-free-included-label {
93+
border-radius: 24px;
94+
}
95+
}

client/my-sites/themes/banners-modern/difm-banner.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ const DIFMBanner = () => {
2222
<p className="banner-modern__description">
2323
{ preventWidows(
2424
translate(
25-
'Built by WordPress.com experts and fully managed for you\u2014no coding or setup required.'
25+
'Launch faster and sell smarter with features designed to save time and delight your customers.'
2626
)
2727
) }
2828
</p>

client/my-sites/themes/banners-modern/style.scss

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,9 @@
1616
@include break-mobile {
1717
max-width: 60%;
1818
}
19+
@include break-large {
20+
max-width: 40%;
21+
}
1922
}
2023

2124
.banner-modern__title {
@@ -75,6 +78,7 @@
7578
var(--ai-builder-banner-linear-gradient-overlay),
7679
url( ./ai-builder-background.png ) no-repeat bottom right -400px / cover,
7780
var(--ai-builder-banner-linear-gradient-underlay);
81+
padding: 40px;
7882
}
7983
@include break-medium {
8084
background:
@@ -170,13 +174,20 @@
170174
padding-bottom: 288px;
171175

172176
@include break-mobile {
173-
background: url( ./difm-background.png ) no-repeat center right 24px / 35%;
177+
background: url( ./difm-background.png ) no-repeat center right 40px / 35%;
174178
background-color: var( --studio-gray-0 );
175-
padding: 24px;
179+
padding: 40px;
176180
}
177-
178181
@include break-medium {
179182
padding-bottom: 72px;
180183
padding-top: 72px;
181184
}
185+
@include break-large {
186+
background: url( ./difm-background.png ) no-repeat center right 40px / 50%;
187+
background-color: var( --studio-gray-0 );
188+
}
189+
@include break-xlarge {
190+
background: url( ./difm-background.png ) no-repeat center right 40px / 40%;
191+
background-color: var( --studio-gray-0 );
192+
}
182193
}

client/my-sites/themes/banners-modern/test/difm-banner.test.tsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,7 @@ describe( 'DIFMBanner', () => {
2222

2323
test( 'renders subtitle', () => {
2424
render( <DIFMBanner /> );
25-
expect(
26-
screen.getByText( /Built by WordPress\.com experts and fully managed for you/ )
27-
).toBeVisible();
25+
expect( screen.getByText( /Launch faster and sell smarter with features/ ) ).toBeVisible();
2826
} );
2927

3028
test( 'renders CTA button linking to DIFM landing page', () => {

client/my-sites/themes/filter-bar-modern/style.scss

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,11 @@
1919
.category-pill-navigation__button {
2020
text-decoration: none;
2121

22+
&:not(.is-active):hover, &:not(.is-active):focus {
23+
background-color: var( --studio-gray-10 );
24+
color: var( --studio-gray-80 );
25+
}
26+
2227
&:first-child {
2328
margin-left: 0;
2429
}
@@ -31,7 +36,7 @@
3136
display: flex;
3237
margin: 0 auto;
3338
max-width: 1220px;
34-
padding: 24px;
39+
padding: 32px 24px;
3540

3641
@media ( min-width: $break-wide ) {
3742
padding-left: 0;

client/my-sites/themes/hero-modern/style.scss

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ $hero-modern-navbar-height: 57px; // UniversalHeaderNavigation
4545
font-weight: 400;
4646
line-height: 1;
4747
margin: 0 0 8px 0;
48+
text-wrap: balance;
4849

4950
@media ( min-width: $break-small ) {
5051
font-size: rem(70px);
@@ -56,6 +57,7 @@ $hero-modern-navbar-height: 57px; // UniversalHeaderNavigation
5657
font-size: $font-body-large;
5758
line-height: 1.5;
5859
margin: 0;
60+
text-wrap: balance;
5961
}
6062

6163
.hero-modern__search {
@@ -70,11 +72,15 @@ $hero-modern-navbar-height: 57px; // UniversalHeaderNavigation
7072
height: 55px;
7173

7274
.components-input-control__container {
75+
.components-input-control__prefix {
76+
color: var(--studio-gray-30);
77+
}
7378
input.components-input-control__input {
79+
color: var(--studio-gray-90);
7480
font-size: $font-body;
7581
}
7682
.components-input-control__backdrop {
77-
border-color: var(--studio-gray-5);
83+
border-color: transparent;
7884
border-radius: 4px;
7985
}
8086
}

client/my-sites/themes/sections-modern/style.scss

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
--theme-section-text: var(--studio-gray-100);
88
--theme-section-text-secondary: var(--studio-gray-80);
99

10-
margin-bottom: 96px;
10+
margin-bottom: 108px;
1111
padding-inline: 24px;
1212

1313
&:last-child {
@@ -21,7 +21,7 @@
2121
--theme-section-text-secondary: var(--studio-gray-10);
2222

2323
background-color: var(--theme-section-bg);
24-
padding: 24px 24px 96px 24px;
24+
padding: 48px 24px 96px 24px;
2525
}
2626

2727
.theme-section-header {
@@ -64,11 +64,13 @@
6464
.theme-section-header__button.components-button.is-secondary {
6565
--wp-components-color-accent: var(--studio-gray-80);
6666
--wp-components-color-accent-darker-20: var(--studio-gray-100);
67+
padding-left: 20px;
68+
padding-right: 20px;
6769
}
6870

6971
.theme-section-modern__grid {
7072
display: grid;
71-
gap: 24px;
73+
gap: 48px;
7274
grid-template-columns: 1fr;
7375
margin: 0 auto;
7476
max-width: 1220px;
@@ -96,7 +98,12 @@
9698
}
9799

98100
.recommended-sections .banner-modern {
99-
margin: 96px auto;
101+
margin: 108px auto;
100102
max-width: calc( 1220px - 48px);
101103
width: calc( 100% - 48px - 48px);
104+
105+
@include break-mobile {
106+
max-width: calc( 1220px - 80px);
107+
width: calc( 100% - 48px - 80px);
108+
}
102109
}

client/my-sites/themes/theme-showcase.scss

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -639,10 +639,10 @@
639639
}
640640

641641
.themes-list {
642+
gap: 48px;
642643
margin: 0 auto;
643644

644645
@include break-small {
645-
gap: 32px;
646646
grid-template-columns: repeat(auto-fill, minmax(320px, 1fr));
647647
max-width: 1220px;
648648
}
@@ -658,6 +658,14 @@
658658
box-shadow: 0 1px 4px 0 rgba(0, 0, 0, 0.25);
659659
}
660660

661+
.theme-card__info {
662+
height: auto;
663+
}
664+
665+
.theme-card__info-title {
666+
font-size: 18px;
667+
}
668+
661669
// Hide style variation swatches for logged-out modern view
662670
.theme-card__info-style-variations {
663671
display: none;

client/my-sites/themes/themes-faq/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ export default function ThemesFAQ() {
155155
<div className="themes-faq">
156156
<div className="themes-faq__wrapper">
157157
<div className="themes-faq__header">
158-
<h2 className="themes-faq__title">{ translate( 'Themes FAQs' ) }</h2>
158+
<h2 className="themes-faq__title">{ translate( 'WordPress theme FAQs' ) }</h2>
159159
</div>
160160
<div className="themes-faq__list">
161161
{ faqItems.map( ( item ) => (

client/my-sites/themes/themes-faq/style.scss

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,11 @@
4646
line-height: 1.2;
4747
color: var( --studio-white );
4848
margin: 0;
49+
text-wrap: balance;
4950

5051
@include break-large {
5152
font-size: rem( 50px );
53+
max-width: 500px;
5254
}
5355
}
5456

0 commit comments

Comments
 (0)