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.

Remaining tasks

User interface changes

API changes

Data model changes

Comments

toncic created an issue. See original summary.

Ginovski’s picture

Title: Add flags into summary function » Allow full or reduced summary
Issue summary: View changes
Related issues: +#2852001: Consider container multivalue children for closed summary

Changing issue name and description accordingly to the missing issue.
Children recursion is covered in #2852001: Consider container multivalue children for closed summary

thenchev’s picture

Assigned: Unassigned » thenchev
Primsi’s picture

Ginovski’s picture

Status: Active » Needs review

Should this be configurable when creating a paragraph type or a global setting that would set a full or reduced summary for all paragraph types?

miro_dietiker’s picture

It'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).

miro_dietiker’s picture

Priority: Normal » Major
Status: Needs review » Active

Major because the summary needs to be better.

Ginovski’s picture

Status: Active » Postponed

This is overlapping with the multivalue summary #2852001: Consider container multivalue children for closed summary, I guess we can postpone it till that gets in.

Berdir’s picture

Status: Postponed » Active
miro_dietiker’s picture

Status: Active » Fixed

The summary has modularity now and we have some contexts where it recurses and others where not.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.