Issue #2775131 by chr.fritsch, marcoscano, katzilla, seanB, xjm, gippy, phenaproxima, alexpott, Berdir, lauriii, webflo, dawehner: Media entities should support contextual links

core/modules/media/templates/media.html.twig

@@ -26,11 +26,16 @@
  * - title_attributes: Same as attributes, except applied to the main title
  *   tag that appears in the template.
  *
+ * Note that we print {{ title_suffix }} even though the title is not included.
+ * This is intentional, once contextual links are embedded in the "title_suffix"
+ * variable. See #2775131 for more information on the background about this.
+ *
  * @see template_preprocess_media()
  *
  * @ingroup themeable
  */
 #}
 <div{{ attributes }}>
+  {{ title_suffix }}
   {{ content }}
 </div>

core/modules/media/tests/src/Functional/MediaContextualLinksTest.php

@@ -0,0 +1,45 @@
+<?php
+
+namespace Drupal\Tests\media\Functional;
+
+use Drupal\media\Entity\Media;
+
+/**
+ * Tests views contextual links on media items.
+ *
+ * @group media
+ */
+class MediaContextualLinksTest extends MediaFunctionalTestBase {
+
+  /**
+   * {@inheritdoc}
+   */
+  public static $modules = [
+    'contextual',
+  ];
+
+  /**
+   * Tests contextual links.
+   */
+  public function testMediaContextualLinks() {
+    // Create a media type.
+    $mediaType = $this->createMediaType();
+
+    // Create a media item.
+    $media = Media::create([
+      'bundle' => $mediaType->id(),
+      'name' => 'Unnamed',
+    ]);
+    $media->save();
+
+    $user = $this->drupalCreateUser([
+      'administer media',
+      'access contextual links',
+    ]);
+    $this->drupalLogin($user);
+
+    $this->drupalGet('media/' . $media->id());
+    $this->assertSession()->elementAttributeContains('css', 'div[data-contextual-id]', 'data-contextual-id', 'media:media=' . $media->id() . ':');
+  }
+
+}

core/themes/classy/templates/content/media.html.twig

@@ -7,6 +7,10 @@
  * - name: Name of the media.
  * - content: Media content.
  *
+ * Note that we print {{ title_suffix }} even though the title is not included.
+ * This is intentional, once contextual links are embedded in the "title_suffix"
+ * variable. See #2775131 for more information on the background about this.
+ *
  * @see template_preprocess_media()
  *
  * @ingroup themeable
@@ -21,6 +25,7 @@
   ]
 %}
 <article{{ attributes.addClass(classes) }}>
+  {{ title_suffix }}
   {% if content %}
     {{ content }}
   {% endif %}

core/themes/stable/templates/content/media.html.twig

@@ -7,12 +7,17 @@
  * - name: Name of the media.
  * - content: Media content.
  *
+ * Note that we print {{ title_suffix }} even though the title is not included.
+ * This is intentional, once contextual links are embedded in the "title_suffix"
+ * variable. See #2775131 for more information on the background about this.
+ *
  * @see template_preprocess_media()
  *
  * @ingroup themeable
  */
 #}
 <article{{ attributes }}>
+  {{ title_suffix }}
   {% if content %}
     {{ content }}
   {% endif %}