-
Notifications
You must be signed in to change notification settings - Fork 843
Forms: Try a horizontal form variation #45234
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.
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
base: trunk
Are you sure you want to change the base?
Conversation
|
Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.
Interested in more tips and information?
|
|
Thank you for your PR! When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:
This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖 Follow this PR Review Process:
If you have questions about anything, reach out in #jetpack-developers for guidance! Jetpack plugin: The Jetpack plugin has different release cadences depending on the platform:
If you have any questions about the release process, please ask in the #jetpack-releases channel on Slack. |
Code Coverage SummaryCoverage changed in 1 file.
Full summary · PHP report · JS report If appropriate, add one of these labels to override the failing coverage check:
Covered by non-unit tests
|
| <> | ||
| <Path fillRule="evenodd" clipRule="evenodd" d="M18 9H13V7.5H18V9Z" /> | ||
| <Path | ||
| fillRule="evenodd" | ||
| clipRule="evenodd" | ||
| d="M9.5 7.5H7.5V9.5H9.5V7.5ZM7.5 6H9.5C10.3284 6 11 6.67157 11 7.5V9.5C11 10.3284 10.3284 11 9.5 11H7.5C6.67157 11 6 10.3284 6 9.5V7.5C6 6.67157 6.67157 6 7.5 6Z" | ||
| /> | ||
| <Path | ||
| fillRule="evenodd" | ||
| clipRule="evenodd" | ||
| d="M19 4.5H5C4.72386 4.5 4.5 4.72386 4.5 5V19C4.5 19.2761 4.72386 19.5 5 19.5H19C19.2761 19.5 19.5 19.2761 19.5 19V5C19.5 4.72386 19.2761 4.5 19 4.5ZM5 3C3.89543 3 3 3.89543 3 5V19C3 20.1046 3.89543 21 5 21H19C20.1046 21 21 20.1046 21 19V5C21 3.89543 20.1046 3 19 3H5Z" | ||
| /> | ||
| </> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: if we move forward with this approach, let's check with Ilona to design an icon.
adb0663 to
5ee7660
Compare
Introduces a new 'horizontal' variation for the contact form block, allowing fields to be displayed side by side. Updates JS logic, SCSS editor styles, and frontend styles to support the horizontal layout, including responsive adjustments for mobile screens. Apply suggestions from code review - use includes instead of indexOf - use "Subscribe" as default button label Co-authored-by: Douglas Henri <dhasilva@users.noreply.github.com> Co-authored-by: Mikael Korpela <mikael@ihminen.org>
26f7658 to
a1e4dd4
Compare
|
This PR has been marked as stale. This happened because:
If this PR is still useful, please do a [trunk merge or rebase](https://github.com/Automattic/jetpack/blob/trunk/docs/git-workflow.md#keeping-your-branch-up-to-date) and otherwise make sure it's up to date and has clear testing instructions. You may also want to ping possible reviewers in case they've forgotten about it. Please close this PR if you think it's not valid anymore — if you do, please add a brief explanation. If the PR is not updated (or at least commented on) in another month, it will be automatically closed. |
This PR makes it easier to create horizontal forms.
By adding a new horizontal form variation.
This is juts an exploration if this is the way we want to go.
Proposed changes:
Other information:
Jetpack product discussion
Does this pull request change what data or activity we track or use?
Testing instructions:
In the editor.
Add a new form. Select the new horizontal form variation.
Does it look as you would expect? Does it load on the front end as you would expect.