Currently a library item paragraph is rendered with a div.paragraph--type--from-library wrapper, and an inner div.field--name-paragraphs wrapper, before rendering the referenced paragraph.

Should it be rendered just as the referenced paragraph, without those wrappers? This patch removes at least the entity reference field wrapper, and sets the outermost wrapper to inherit the classes of the referenced paragraph type. It still results in 2 paragraph wrappers, any ideas how to remove that?

CommentFileSizeAuthor
#2 paragraphs_collection-library_remove_wrapper-2923815-1.patch775 bytesbgilhome
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bgilhome created an issue. See original summary.

bgilhome’s picture

bgilhome CreditAttribution: bgilhome commented
FileSize
paragraphs_collection-library_remove_wrapper-2923815-1.patch775 bytes
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Project: Paragraphs Collection » Paragraphs
Component: Library » Module: Library

Moving to Paragraphs project.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented

Maybe we still should pass in the original item into the yml so access is still possible?

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Needs review » Needs work
Issue tags: +Needs tests

And we should have tests for this to correctly validate and document the expectations.