-
Notifications
You must be signed in to change notification settings - Fork 87
feat(FormLayout): add FormRow and br support to autoResponsive mode #8699
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
feat(FormLayout): add FormRow and br support to autoResponsive mode #8699
Conversation
ba43675 to
a618c26
Compare
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.
Currently, the component creates more columns than necessary. This is planned to be addressed within #8695.
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.
One NIT and one clarification comment so far.
add entrypoints, screenshot tests minimize diff implement br support add snapshot tests update snapshot tests add more tests
Co-authored-by: Diego Cardoso <diego@vaadin.com>
ba305c9 to
a056bff
Compare
|
|
This ticket/PR has been released with Vaadin 24.8.0.rc2 and is also targeting the upcoming stable 24.8.0 version. |



Description
<vaadin-form-row>for arranging fields into rows<br>to provide the ability to force a new row when using autoRows.Fixes #8686
Type of change