- Issue created by @ericgsmith
- Status changed to Needs review
about 1 year ago 3:25am 1 November 2023
✨ Adding wrapper for the summary Fixed added a wrapper to the summary by hard coding a div with a class on it into the markup.
This is hard to remove as its being added to the text value of the field - rather than something like an attribute or wrapper on the item.
E.g this is problematic for my use case - I use this formatter in a views data export to get summary values, and then expect to be able to re import the data at a later stage in a different process without have to either manually strip tags / remove the summary wrapper.
I'm not sure what the original intent was with the issue - if the field does not have a summary then nothing is rendered, if it has a summary then something is rendered. You can use the standard field classes / theme preprocessing to add additional classes? I don't think it should be up to this formatted to provide anything additional to the theme layer.
Without understanding the issue further I don't have any other suggestions - but I would prefer to revert ✨ Adding wrapper for the summary Fixed
Needs review
1.0
Code