Skip to content

Forms: Use Core buttons instead of Jetpack buttons #45007

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 16 commits into
base: trunk
Choose a base branch
from
Open

Forms: Use Core buttons instead of Jetpack buttons #45007

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
57725d7
Switch contact form submit button to core/buttons block
lezama Aug 28, 2025
8066c25
simpler
lezama Aug 28, 2025
05e4398
Restrict flex styling to specific wp-block elements
lezama Aug 28, 2025
bed35f0
Add spinner animation for submitting form button
lezama Aug 28, 2025
dfdbd59
Restrict tag processing to button and update submit styles
lezama Aug 28, 2025
57e1c87
changelog
lezama Aug 28, 2025
80d8670
Update projects/packages/forms/src/blocks/contact-form/variations.js
lezama Aug 29, 2025
00d63f9
Refactor render_submit_button to improve clarity
lezama Aug 29, 2025
3f1849f
Update projects/packages/forms/src/blocks/contact-form/variations.js
lezama Sep 15, 2025
7aee2f9
add some tests to improve coverage
CGastrell Sep 16, 2025
46c0779
fix disabled and submitting states
CGastrell Sep 16, 2025
12c24bb
update tests after attribute addition
CGastrell Sep 16, 2025
c7684da
modify useformwrapper to use core/buttons as well
CGastrell Sep 16, 2025
2bfc41f
Apply suggestion from @manzoorwanijk
lezama Sep 17, 2025
7102244
Update projects/packages/forms/src/blocks/contact-form/class-contact-…
lezama Sep 17, 2025
fac10d5
change patterns to use core buttons
CGastrell Sep 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
change patterns to use core buttons
  • Loading branch information
@CGastrell
CGastrell committed Nov 28, 2025
commit fac10d5e0ba69cc2a128164111ff8d6597b27f39
49 changes: 42 additions & 7 deletions projects/packages/forms/src/contact-form/class-util.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,12 @@ public static function register_pattern() {
<!-- wp:jetpack/field-name {"required":true} /-->
<!-- wp:jetpack/field-email {"required":true} /-->
<!-- wp:jetpack/field-textarea /-->
<!-- wp:jetpack/button {"element":"button","text":"Contact Us","lock":{"remove":true}} /-->
<!-- wp:buttons {"lock":{"move":false,"remove":true}} -->
<div class="wp-block-buttons">
<!-- wp:button {"tagName":"button","type":"submit","className":"jetpack-form-submit-button"} -->
<div class="wp-block-button jetpack-form-submit-button"><button type="submit" class="wp-block-button__link wp-element-button">Contact us</button></div>
<!-- /wp:button --></div>
<!-- /wp:buttons -->
</div>
<!-- /wp:jetpack/contact-form -->',
),
Expand All @@ -69,7 +74,12 @@ public static function register_pattern() {
<!-- wp:jetpack/field-name {"required":true} /-->
<!-- wp:jetpack/field-email {"required":true} /-->
<!-- wp:jetpack/field-consent /-->
<!-- wp:jetpack/button {"element":"button","text":"Subscribe","lock":{"remove":true}} /-->
<!-- wp:buttons {"lock":{"move":false,"remove":true}} -->
<div class="wp-block-buttons">
<!-- wp:button {"tagName":"button","type":"submit","className":"jetpack-form-submit-button"} -->
<div class="wp-block-button jetpack-form-submit-button"><button type="submit" class="wp-block-button__link wp-element-button">Subscribe</button></div>
<!-- /wp:button --></div>
<!-- /wp:buttons -->
</div>
<!-- /wp:jetpack/contact-form -->',
),
Expand All @@ -83,7 +93,12 @@ public static function register_pattern() {
<!-- wp:jetpack/field-email {"required":true} /-->
<!-- wp:jetpack/field-radio {"label":"Attending?","required":true,"options":["Yes","No"]} /-->
<!-- wp:jetpack/field-textarea {"label":"Other Details"} /-->
<!-- wp:jetpack/button {"element":"button","text":"Send RSVP","lock":{"remove":true}} /-->
<!-- wp:buttons {"lock":{"move":false,"remove":true}} -->
<div class="wp-block-buttons">
<!-- wp:button {"tagName":"button","type":"submit","className":"jetpack-form-submit-button"} -->
<div class="wp-block-button jetpack-form-submit-button"><button type="submit" class="wp-block-button__link wp-element-button">Send RSVP</button></div>
<!-- /wp:button --></div>
<!-- /wp:buttons -->
</div>
<!-- /wp:jetpack/contact-form -->',
),
Expand All @@ -98,7 +113,12 @@ public static function register_pattern() {
<!-- wp:jetpack/field-telephone {"label":"Phone Number"} /-->
<!-- wp:jetpack/field-select {"label":"How did you hear about us?","options":["Search Engine","Social Media","TV","Radio","Friend or Family"]} /-->
<!-- wp:jetpack/field-textarea {"label":"Other Details"} /-->
<!-- wp:jetpack/button {"element":"button","text":"Send","lock":{"remove":true}} /-->
<!-- wp:buttons {"lock":{"move":false,"remove":true}} -->
<div class="wp-block-buttons">
<!-- wp:button {"tagName":"button","type":"submit","className":"jetpack-form-submit-button"} -->
<div class="wp-block-button jetpack-form-submit-button"><button type="submit" class="wp-block-button__link wp-element-button">Send</button></div>
<!-- /wp:button --></div>
<!-- /wp:buttons -->
</div>
<!-- /wp:jetpack/contact-form -->',
),
Expand All @@ -114,7 +134,12 @@ public static function register_pattern() {
<!-- wp:jetpack/field-date {"label":"Date","required":true} /-->
<!-- wp:jetpack/field-radio {"label":"Time","required":true,"options":["Morning","Afternoon"]} /-->
<!-- wp:jetpack/field-textarea {"label":"Notes"} /-->
<!-- wp:jetpack/button {"element":"button","text":"Book Appointment","lock":{"remove":true}} /-->
<!-- wp:buttons {"lock":{"move":false,"remove":true}} -->
<div class="wp-block-buttons">
<!-- wp:button {"tagName":"button","type":"submit","className":"jetpack-form-submit-button"} -->
<div class="wp-block-button jetpack-form-submit-button"><button type="submit" class="wp-block-button__link wp-element-button">Book Appointment</button></div>
<!-- /wp:button --></div>
<!-- /wp:buttons -->
</div>
<!-- /wp:jetpack/contact-form -->',
),
Expand All @@ -131,7 +156,12 @@ public static function register_pattern() {
<!-- wp:jetpack/input-rating /--></div>
<!-- /wp:jetpack/field-rating -->
<!-- wp:jetpack/field-textarea {"label":"How could we improve?"} /-->
<!-- wp:jetpack/button {"element":"button","text":"Send Feedback","lock":{"remove":true}} /-->
<!-- wp:buttons {"lock":{"move":false,"remove":true}} -->
<div class="wp-block-buttons">
<!-- wp:button {"tagName":"button","type":"submit","className":"jetpack-form-submit-button"} -->
<div class="wp-block-button jetpack-form-submit-button"><button type="submit" class="wp-block-button__link wp-element-button">Send Feedback</button></div>
<!-- /wp:button --></div>
<!-- /wp:buttons -->
</div>
<!-- /wp:jetpack/contact-form -->',
),
Expand All @@ -147,7 +177,12 @@ public static function register_pattern() {
<!-- wp:jetpack/field-telephone {"label":"Phone","id":"phone"} /-->
<!-- wp:jetpack/field-text {"label":"Company","id":"company"} /-->
<!-- wp:jetpack/field-text {"label":"Job Title","id":"title"} /-->
<!-- wp:jetpack/button {"element":"button","text":"Submit","lock":{"remove":true}} /-->
<!-- wp:buttons {"lock":{"move":false,"remove":true}} -->
<div class="wp-block-buttons">
<!-- wp:button {"tagName":"button","type":"submit","className":"jetpack-form-submit-button"} -->
<div class="wp-block-button jetpack-form-submit-button"><button type="submit" class="wp-block-button__link wp-element-button">Submit</button></div>
<!-- /wp:button --></div>
<!-- /wp:buttons -->
</div>
<!-- /wp:jetpack/contact-form -->',
),
Expand Down
Loading