Problem/Motivation
Olivero:

Bootstrap 3:

UI Suite Bootstrap 5:

Details:
- panel in Bootstrap 3 https://getbootstrap.com/docs/3.4/components/#panels
- card + collapse from Bootstrap 4 https://getbootstrap.com/docs/4.6/migration/#panels
Regarding the look in Olivero:
- Fieldsets => Card
- Details => Accordion
| Comment | File | Size | Author |
|---|---|---|---|
| #21 | Capture d’écran de 2023-06-02 16-52-31.png | 51.73 KB | grimreaper |
| #12 | Capture d’écran de 2023-05-25 18-14-26.png | 31.47 KB | grimreaper |
| #12 | Capture d’écran de 2023-05-25 18-10-03.png | 66.3 KB | grimreaper |
| #9 | Capture d’écran de 2023-05-25 17-46-57.png | 20.27 KB | grimreaper |
| #8 | Capture d’écran de 2023-05-25 12-26-24.png | 33.21 KB | grimreaper |
Issue fork ui_suite_bootstrap-3345265
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
grimreaperComment #3
grimreaperComment #4
grimreaperComment #5
grimreaperComment #7
grimreaperEven if Drupal does not have "inline" label for fieldset as of https://getbootstrap.com/docs/5.2/forms/layout/#horizontal-form, I will try to support that.
Comment #8
grimreaperComment #9
grimreaperFinally, I removed the core library as its target details and summary and details.html.twig no more output those tags.
I was not able to call the accordion and accordion_item patterns because of the title and content fields. Not possible to prepare HTML to inject to those fields, it is otherwise escaped.
If someone has a tip on that, I will be happy that it is shared.
Comment #10
grimreaperComment #11
grimreaperFixing default open state.
Comment #12
grimreaperTested on my project with Drupal Commerce.
Comment #13
grimreaperI need to test with forms in Layout Builder (offanvas)
Comment #14
grimreaperThat's what I thought. It breaks Layout Builder offcanvas forms.
When using gin_lb it is ok because gin_lb provides template suggestions so the presenter template are not taken into account.
I will check if possible to do something similar by default.
Comment #15
grimreaperComment #16
grimreaperComment #17
grimreaperComment #18
grimreaperCommerce user registration step is no more styled.
The problem is that the fieldset of this form needs to be wrapped in a div that will take the col-md-6 class and no more the fieldsets directly.
Comment #19
grimreaperComment #20
grimreaperComment #21
grimreaperComment #23
grimreaperComment #30
grimreaper