Problem/Motivation
Folow-up from #2857112: Add summary method into paragraph entity
We discuss here.
The reduced version will skip elements that have an icon, covered in the ui already.
-That is now and likely also in the future in the UI somehow. also doesn't actually seem to be in the summary despite being in the code, will check that. But again, letting code decide that is tricky, just because we *support* icons doesn't mean everyone will have time, so possibly a setting instead. But we can also add a currently empty $context or $options array and we'll add options through that in the future. then we're flexible.
-A pattern about summary could be that the (tbd) icon methods will return icon definitions with keys. Then we could pass in options to the summary with some skip key that gets the icon keys that are displayed otherwise. The summary then excludes them on creation.
Proposed resolution
A flag if it should be full or reduced.
Comments
Comment #2
Ginovski CreditAttribution: Ginovski at MD Systems GmbH commentedChanging issue name and description accordingly to the missing issue.
Children recursion is covered in #2852001: Consider container multivalue children for closed summary
Comment #3
thenchev CreditAttribution: thenchev commentedComment #4
Primsi CreditAttribution: Primsi at MD Systems GmbH for MD Systems GmbH commentedComment #5
Ginovski CreditAttribution: Ginovski at MD Systems GmbH commentedShould this be configurable when creating a paragraph type or a global setting that would set a full or reduced summary for all paragraph types?
Comment #6
miro_dietikerIt's not configurable.
We have (or will have) multiple locations where the paragraphs are displayed as summary.
Some locations will show the summary with recursion, others without (because they display the children differently).
Comment #7
miro_dietikerMajor because the summary needs to be better.
Comment #8
Ginovski CreditAttribution: Ginovski at MD Systems GmbH commentedThis is overlapping with the multivalue summary #2852001: Consider container multivalue children for closed summary, I guess we can postpone it till that gets in.
Comment #9
BerdirComment #10
miro_dietikerThe summary has modularity now and we have some contexts where it recurses and others where not.