- Issue created by @Anybody
- Assigned to Grevil
- First commit to issue fork.
- 🇩🇪Germany Grevil
Yea, that part also doesn't make much sense, as we already received the classes anyway. We basically add them again.
- Merge request !38Issue #3464653 by anybody: Do we miss explode() calls for fences_field_classes, fences_label_classes, fences_field_items_wrapper_classes → (Merged) created by Grevil
- 🇩🇪Germany Anybody Porta Westfalica
Yeah looks totally correct, additionally I think we need to also explode in
https://git.drupalcode.org/issue/fences-3464644/-/blob/3464644-add-a-mul...$vars['field_items_wrapper_attributes'] = new Attribute(); if (!empty($fences_config['fences_field_items_wrapper_classes'])) { $vars['field_items_wrapper_attributes']->addClass($fences_config['fences_field_items_wrapper_classes']); }
Then we should have them all?
- 🇩🇪Germany Grevil
All done, please review! I did it in the Web-IDE, but don't think there are any errors.
- 🇩🇪Germany Anybody Porta Westfalica
Great, tests are green and looks good to me!
Automatically closed - issue fixed for 2 weeks with no activity.