In some cases showing the paragraph_bundle_title field would be relevant or confusing for the user, so we should make it configurable.

CommentFileSizeAuthor
#7 2976842-7.patch3.24 KBedurenye
#7 interdiff-2976842-2-7.txt598 bytesedurenye
#2 2976842-1.patch3.19 KBedurenye
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

edurenye created an issue. See original summary.

edurenye’s picture

edurenye CreditAttribution: edurenye at Digitalist commented
Status: Active » Needs review
FileSize
2976842-1.patch3.19 KB

Done.

Status: Needs review » Needs work

The last submitted patch, 2: 2976842-1.patch, failed testing. View results

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented

Thank you for the proposal.

Be aware that we are currently D8 first (with strict test coverage) and only backport critical features to D7.
With Drupal 8 we reduced the amount of settings and try to keep them minimal... although they're permanently growing...
But not sure if we can avoid such a setting somehow.

edurenye’s picture

edurenye CreditAttribution: edurenye at Digitalist commented

Hi Miro,
I needed it for a project and wanted to share it, so others can just apply this patch.
Anyway I think is something interesting to have in d7.
For d8 we should wait for this #2353867: [META] Expose Title and other base fields in Manage Display and make sure that when this is in our title works as expected. What do you think?

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented

Makes sense. It's not clear to me though if the other issue will also cover the form... The title only refers to "manage display".

edurenye’s picture

edurenye CreditAttribution: edurenye at Digitalist commented
FileSize
interdiff-2976842-2-7.txt598 bytes
2976842-7.patch3.24 KB

Small fix to the patch to avoid errors when using features with a dependency with paragraphs.

james.williams’s picture

james.williams CreditAttribution: james.williams at ComputerMinds commented
Related issues: +#2793711: Handle paragraphs better when only one type available

#2793711: Handle paragraphs better when only one type available has a slightly different approach to hiding this paragraph type title info, in that it only hides it when there is only one paragraph type that can be used in a field. I wonder if this is actually the most common reason for wanting to hide it? I would prefer that approach to configuring on a per-paragraph-type basis, as if the paragraph type might be usable elsewhere alongside other paragraph types, I would then want its type title to show there (but remain hidden where it is the only paragraph type allowed).

Either way - I think we should only choose one or the other of those two approaches, I don't think both should be combined.

Of course, as this is for D7, I doubt either will progress anyway!