- Issue created by @smustgrave
-
smustgrave β
committed 91ac35ba on 4.0.x
Issue #3486199: Audit components before go live
-
smustgrave β
committed 91ac35ba on 4.0.x
- πΊπΈUnited States smustgrave
Updated the project page to include icons.
- π«π·France pdureau Paris
Hello Steven,
A few feedbacks about the current state of 4.x branch
Form components
Some components implement form element. Most SDC themes don't do that because SDC is not compatible with the Form API ( π Compatibility between SDC and the Form API Active ). In UI Suite USWDS, we have a few of them, and there is nothign wrogn about that.
However, some don't keep slots opaque and manipulate renderables in the template.
For example, in
combo_box
:{% if option.type == 'optgroup' or option['#type'] == 'optgroup'%} {% set label = option.label ? option.label : option['#label'] %}
Is this component used somewhere? How? Can we prevent this behaviour?
Layout Options is not necessary anymore
UI Patterns 2.x is also replacing Layout Options:
Proposed resolution
- Create 4 components: grid_row_1, grid_row_2, grid_row_3 & grid_row_4
- Remove layout_options from info.yml & composer.json
- Remove ui_suite_uswds.layout_options.yml and ui_suite_uswds.layouts.yml
- Remove template/layout--grid.html.twig
Example of similar change: π [1.1.0] From Layout Options to UI Patterns 2.x Active
landing_page ui_examples
You have to convert it to UI Patterns 2, to replace:
pattern render
elements bycomponent
render elementspattern_preview
render elements bycomponent
render elements, with astory
render property
It is an emergency to do it IMO, because you have already removed the dependency to
ui_patterns_legacy
This will also be the opportunity to:
- Remove the # prefix in the render property:
"#type": html_tag
andtype: html_tag
- Use the newly introduced Grid components instead of
markup
renderable like"#markup": '<div class="grid-row grid-gap"> <div class="tablet:grid-col-4">...'
Tests with UI Patterns 2
Have you already tried using
ui_suite_uswds
with UI Patterns 2 is projects? That's the best test to do ;) - πΊπΈUnited States smustgrave
Fixed the layout_options/new grid patterns in 1 issue
the ui_example in anotherand the combo box here. Not thrilled with it but that's my fault for being eager with the patterns for forms.
-
smustgrave β
committed 28d6f64c on 4.0.x
Resolve #3486199 "Fix combo box"
-
smustgrave β
committed 28d6f64c on 4.0.x
- Assigned to pdureau
- Status changed to Fixed
4 months ago 10:14pm 9 January 2025 Automatically closed - issue fixed for 2 weeks with no activity.