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
+496 −63
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
Next Next commit
modify useformwrapper to use core/buttons as well
  • Loading branch information
@CGastrell
CGastrell committed Nov 28, 2025
commit c7684da933b77d673645163cc0ef92ac48c73260
13 changes: 8 additions & 5 deletions projects/packages/forms/src/blocks/shared/hooks/use-form-wrapper.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,14 @@ export default function useFormWrapper( { attributes, clientId, name } ) {
clientId,
createBlock( FORM_BLOCK_NAME, {}, [
createBlock( name, attributes, getBlocks( clientId ) ),
createBlock( 'jetpack/button', {
text: __( 'Submit', 'jetpack-forms' ),
element: 'button',
lock: { remove: true },
} ),
createBlock( 'core/buttons', { lock: { remove: true } }, [
createBlock( 'core/button', {
text: __( 'Submit', 'jetpack-forms' ),
type: 'submit',
tagName: 'button',
className: 'wp-block-jetpack-button jetpack-form-submit-button',
} ),
] ),
] )
);
}
Expand Down