HPC-9916: First draft of meta data in diff displays

UN-OCHA · Dec 2, 2024 · 2f76a79 · 2f76a79
1 parent e13133e
commit 2f76a79
Show file tree

Hide file tree

Showing 5 changed files with 80 additions and 3 deletions.
diff --git a/html/modules/custom/ncms_ui/css/diff.css b/html/modules/custom/ncms_ui/css/diff.css
@@ -1,12 +1,25 @@
 .ncms-node-diff #main-content {
   padding-block-start: 0;
 }
-.ncms-node-diff .diff-header {
+.ncms-node-diff .diff-header,
+.ncms-node-diff .metadata-wrapper {
   margin-bottom: 1rem;
   padding-bottom: 1rem;
   border-bottom: 1px solid var(--cd-blue-grey--dark);
 }
 
+.ncms-node-diff .diff-header label,
+.ncms-node-diff .metadata-wrapper label {
+  font-weight: bold;
+}
+
+.ncms-node-diff .diff-header .diff-revision__items .diff-revision__items-group:first-child {
+  background-color: #cfc;
+}
+.ncms-node-diff .diff-header .diff-revision__items .diff-revision__items-group:last-child {
+  background-color: #fcc;
+}
+
 .ncms-node-diff .diff-controls {
   display: none;
 }
@@ -19,3 +32,18 @@
   margin: 2rem 0;
   display: block;
 }
+
+.ncms-node-diff .metadata-wrapper .field--label-inline > .field__label,
+.ncms-node-diff .metadata-wrapper .field--label-inline > .field__item {
+  display: inline;
+}
+.ncms-node-diff .metadata-wrapper .field--label-inline > .field__label {
+  font-weight: bold;
+}
+.ncms-node-diff .metadata-wrapper .field--label-inline > .field__label::after {
+  content: ":";
+}
+.ncms-node-diff .metadata-wrapper .field--type-text-long .field__item p {
+  display: inline;
+  margin-left: 0.3rem;
+}
diff --git a/html/modules/custom/ncms_ui/src/Entity/Content/ContentBase.php b/html/modules/custom/ncms_ui/src/Entity/Content/ContentBase.php
@@ -340,6 +340,41 @@ public function preSave(EntityStorageInterface $storage) {
     $this->setRevisionTranslationAffectedEnforced(TRUE);
   }
 
+  /**
+   * {@inheritdoc}
+   */
+  public function buildMetaDataForDiff() {
+    $build = [
+      '#type' => 'container',
+      '#attributes' => [
+        'class' => ['metadata-wrapper'],
+      ],
+    ];
+    $build['header'] = [
+      '#type' => 'label',
+      '#title' => $this->t('Meta data'),
+      '#title_display' => 'before',
+    ];
+
+    $meta_fields = [
+      'field_short_title',
+      'field_computed_tags',
+      'field_automatically_visible',
+      'field_summary',
+      'field_author',
+      'field_pdf',
+    ];
+    foreach ($meta_fields as $field_name) {
+      if (!$this->hasField($field_name)) {
+        continue;
+      }
+      $build[$field_name] = $this->get($field_name)->view([
+        'label' => 'inline',
+      ]);
+    }
+    return $build;
+  }
+
   /**
    * Get the route match service.
    *

diff --git a/html/modules/custom/ncms_ui/src/Entity/ContentInterface.php b/html/modules/custom/ncms_ui/src/Entity/ContentInterface.php
@@ -43,4 +43,12 @@ public function isDeleted();
    */
   public function getEntityOperations();
 
+  /**
+   * Build the metadata used for diff displays.
+   *
+   * @return array
+   *   A render array.
+   */
+  public function buildMetaDataForDiff();
+
 }
diff --git a/html/themes/custom/common_design_subtheme/templates/nodes/node--article--full.html.twig b/html/themes/custom/common_design_subtheme/templates/nodes/node--article--full.html.twig
@@ -102,6 +102,9 @@
   ]
 %}
 <article{{ attributes.addClass(classes) }}>
+  {% if content.diff_metadata %}
+    {{ content.diff_metadata  }}
+  {% endif %}
   <header class="gho-article__header">
     {{ hero_image }}
     <div class="gho-article__heading content-width content-width--reading">
@@ -115,7 +118,7 @@
   {% endif %}
 
   <div{{ content_attributes.addClass('node__content').addClass('gho-article__content') }}>
-    {{ content|without(['field_hero_image', 'field_appeals']) }}
+    {{ content|without(['field_hero_image', 'field_appeals', 'diff_metadata']) }}
   </div>
 
 </article>
diff --git a/html/themes/custom/common_design_subtheme/templates/nodes/node--document--full.html.twig b/html/themes/custom/common_design_subtheme/templates/nodes/node--document--full.html.twig
@@ -102,6 +102,9 @@
   ]
 %}
 <article{{ attributes.addClass(classes) }}>
+  {% if content.diff_metadata %}
+    {{ content.diff_metadata  }}
+  {% endif %}
   <header class="gho-article__header">
     {{ hero_image }}
     <div class="gho-article__heading content-width content-width--reading">
@@ -114,7 +117,7 @@
   {% endif %}
 
   <div{{ content_attributes.addClass('node__content').addClass('gho-article__content') }}>
-    {{ content|without(['field_hero_image']) }}
+    {{ content|without(['field_hero_image', 'diff_metadata']) }}
   </div>
 
 </article>