Skip to content

Commit

Permalink
Theme: Use block attributes for Jetpack sub form styling where possible
Browse files Browse the repository at this point in the history
See #2828
  • Loading branch information
adamwoodnz committed Aug 29, 2024
1 parent 19c36fd commit 243564a
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 22 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@
<p class="has-small-font-size"><?php esc_html_e( 'Get the latest news about everything Learn WordPress. Find out when new courses and lessons are available, and when registration opens for upcoming Online Workshops.', 'wporg-learn' ); ?></p>
<!-- /wp:paragraph -->

<!-- wp:jetpack/subscriptions {"className":"is-style-compact"} /-->
<!-- wp:jetpack/subscriptions {"borderRadius":2,"borderWeight":0,"className":"is-style-compact"} /-->

</div>
<!-- /wp:column -->
Expand Down
20 changes: 6 additions & 14 deletions wp-content/themes/pub/wporg-learn-2024/src/style/_jetpack.scss
Original file line number Diff line number Diff line change
@@ -1,19 +1,11 @@
// Align the subscribed button to the left
/* stylelint-disable-next-line max-line-length */
.wp-block-jetpack-subscriptions.wp-block-jetpack-subscriptions__supports-newline:not(.wp-block-jetpack-subscriptions__use-newline) {
#subscribe-email input {
border-top-left-radius: 2px !important;
border-bottom-left-radius: 2px !important;
}

p#subscribe-submit {
justify-content: start;
.wp-block-jetpack-subscriptions.wp-block-jetpack-subscriptions__supports-newline:not(.wp-block-jetpack-subscriptions__use-newline) p#subscribe-submit {
justify-content: flex-start;
}

button {
border-top-right-radius: 2px !important;
border-bottom-right-radius: 2px !important;
font-weight: 600;
}
}
.wp-block-jetpack-subscriptions .wp-block-button__link {
font-weight: 600;
}

.wp-block-jetpack-contact-form-container .contact-form {
Expand Down
7 changes: 0 additions & 7 deletions wp-content/themes/pub/wporg-learn-2024/src/style/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,6 @@ body {
}

.wporg-learn-footer-signup {
input[type="email"] {
border-color: var(--wp--custom--form--color--background);
}

button[type="submit"] {
font-weight: 400;
}

@media screen and (max-width: 781px) {
border-left: none;
Expand Down

0 comments on commit 243564a

Please sign in to comment.