Profile entities should be rendered together with contextual links for consistency with the general editor flow for entities.

This is fixed by defining the links in profile.links.contextual.yml, adding the links via ProfileViewBuilder::alterBuild() and rendering them via $title_suffix.

CommentFileSizeAuthor
#15 Screen Shot 2019-03-07 at 2.47.43 PM.png44.05 KBmglaman
#14 2922979-14.patch991 bytesmglaman
#11 profile-support_contextual_links-2922979-11.patch569 bytesszeidler
#3 profile-contextual_links-2922979-3.patch1.59 KBolofbokedal
#2 profile-contextual_links-2922979-2.patch1.19 KBolofbokedal
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

olofjohansson created an issue. See original summary.

olofbokedal’s picture

olofbokedal CreditAttribution: olofbokedal at Odd Hill for European Spallation Source commented
FileSize
profile-contextual_links-2922979-2.patch1.19 KB

Patch implements the steps mentioned in the issue description.

olofbokedal’s picture

olofbokedal CreditAttribution: olofbokedal at Odd Hill for European Spallation Source commented
FileSize
profile-contextual_links-2922979-3.patch1.59 KB

First patch didn't include new file. Fixed.

zerolab’s picture

zerolab CreditAttribution: zerolab at Torchbox for The Chartered Society of Physiotherapy commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 3: profile-contextual_links-2922979-3.patch, failed testing. View results

olofbokedal’s picture

olofbokedal CreditAttribution: olofbokedal at Odd Hill for European Spallation Source commented

error: profile.links.contextual.yml: No such file or directory according to the failed test.

This file should get created. Anyone knows how to avoid this error? Patch applied cleanly for me but perhaps it needs to be created in a different way.

maaty388’s picture

maaty388 CreditAttribution: maaty388 commented

I believe it has to do something with tests, correct me if I am wrong. Your patch is working fine, gets applied and everything so this is the only thing that I can think of right now.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
CreditAttribution: jwilson3 at Bluespark commented
Status: Needs work » Needs review

trigger tests again maybe?

Status: Needs review » Needs work

The last submitted patch, 3: profile-contextual_links-2922979-3.patch, failed testing. View results

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented

You'll want to retest this on Drupal 8.5.0, which now automatically adds contextual links, making at least the ViewBuilder portion of the patch unnecessary.

szeidler’s picture

szeidler CreditAttribution: szeidler at Ramsalt Lab commented
Status: Needs work » Needs review
FileSize
profile-support_contextual_links-2922979-11.patch569 bytes

With Drupal 8.5.0 and https://www.drupal.org/node/2928555 it seems, that only {{ title_suffix }} is required for making the contextual links working for me. Could you review the reduced patch?

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented 
    'title_prefix' => [],
    'title_suffix' => [],

Prefix and suffix are defaults. May as well add them both, even we are not rendering the profile label/title separate.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented

Nevermind, here is what media.html.twig has

<div{{ attributes }}>
  {{ title_suffix.contextual_links }}
  {{ content }}
</div>

Let's just copy them. They prove the existing patch is all we need.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented
FileSize
2922979-14.patch991 bytes

Here is #11 with the links from #3 (otherwise nothing rendered.)

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented
FileSize
Screen Shot 2019-03-07 at 2.47.43 PM.png44.05 KB

Attaching a screenshot for good measure.

  • mglaman committed 5400ceb on 8.x-1.x 
    Issue #2922979 by olofbokedal, mglaman, szeidler: Implement contextual...
mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented
Status: Needs review » Fixed

Committed! Thanks, everyone.

Status: Fixed » Closed (fixed)

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