Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.In the Drupal 8 item-list.html.twig there is an list_style variable, this is not set in the build function of a facet in WidgetPluginBase.php on line 63. Via the context array its possible to add the list style. The list style is set by an setting in a facet. See attached patch for a fixed. Would love a review.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | facets-list_style_context-2841041-5.patch | 781 bytes | jefuri |
| #2 | list-style-2841041.patch | 765 bytes | markkes |
Comments
Comment #2
markkes CreditAttribution: markkes at Synetic commentedComment #3
borisson_Thanks for creating this issue, I have no idea what this does but I'll have a look at it soon.
Comment #4
markkes CreditAttribution: markkes at Synetic commentedBy setting the list_style via the context in the build of the item-list the class gets passed into the twig template. See: core/modules/system/templates/item-list.html.twig on line 25. Check it out, would like to hear what you think.
Comment #5
jefuri CreditAttribution: jefuri as a volunteer and commentedRecreated the patch from markkes by using the correct drupal patch flow for contribs.
Comment #8
borisson_Thanks! Committed.