-
Notifications
You must be signed in to change notification settings - Fork 419
refactor: Improve spacing behavior for nested layouts with conditional visibility #2533
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: master
Are you sure you want to change the base?
Conversation
…p layouts now do not have extra paddings
…add up, also fixed the issue where some of the children can be hidden for which no padding is added
…remove extra padding
✅ Deploy Preview for jsonforms-examples ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
@sdirix @lucas-koehler please review - new demo example employee-registration |
|
found an issue with horizontal layout when we do not use padding but gap in order to fix the issue with the elements that are invisible because of the security to not have extra space but this breaks when we want to use number of columns in the UI model like |
…ut can't be easily fixed with gap and allowing the v-col cols property to work
|
fixing the horizontal and vertical layout by using no-gutters for vertical and fluid for both horizontal and vertical layouts |
…nvisible elements to not have extra padding from the v-col
…ements that are not visible.
|
fix removing extra paddings for invisible elements by not creating the v-col for such elements. |
…not have v-container paddings
visible: falseMake the group layout and array layouts/control to use collapsable panels to make it easy to deal with array with large number of elements if we want to collapse the whole array UI into just one row.
Make the space usage similar to react - e.g. remove exta margins and/or paddings
Other minor fixes