From d361270f0cda6c496a37b56b42737040bcfd2e22 Mon Sep 17 00:00:00 2001
From: Konstantinos Maninakis <maninak@protonmail.com>
Date: Sat, 9 Nov 2024 02:59:38 +0100
Subject: [PATCH] feat(patch-detail): don't show commit details in major events
 section

Signed-off-by: Konstantinos Maninakis <maninak@protonmail.com>
---
 CHANGELOG.md                                     | 3 ++-
 src/webviews/src/components/PatchMajorEvents.vue | 8 ++------
 2 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index fcb7e1a3..bab06aa9 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -35,6 +35,7 @@
   - the aforementioned coupled with the pre-existing feature of optimally auto-aligning the form as it resizes should seamlessly provide a smooth authoring experience
   - adjust the hover text of all form buttons to advertise their respective keyboard shortcut (if any)
   - **patch-detail:** preselect the merged revision, if any, in the revision selector instead of always the latest
+  - **patch-detail:** don't mention the related commit in the patch major events section (~~"Last updated by W with revision X at commit Y Z time ago"~~). This information is still available in the Revision section.
   - **patch-detail:** keep the labels of the main buttons (top-right of the view) as a single line of text, even if the viewport is narrow enough to compress them
   - **patch-detail:** add themed styling to `<summary>` elements when tabbed into
 
@@ -42,7 +43,7 @@
 
 - **patch-detail:** don't disappear the Activity and Revision sections sometimes, e.g. when a new revision is detected
 - **patch-detail:** make "Refresh" button work again, fetching latest patch data
-- **patch-detail:** don't show extra gap between patch title and next section if description is empty
+- **patch-detail:** don't show big gap between patch title and next section if the patch description is empty
 
 ### 🏡 Chores
 
diff --git a/src/webviews/src/components/PatchMajorEvents.vue b/src/webviews/src/components/PatchMajorEvents.vue
index f705c81a..036c4673 100644
--- a/src/webviews/src/components/PatchMajorEvents.vue
+++ b/src/webviews/src/components/PatchMajorEvents.vue
@@ -40,10 +40,7 @@ setInterval(() => {
       <pre :title="latestMerge.author.id">{{ getIdentityAliasOrId(latestMerge.author) }}</pre>
       using revision
       <pre :title="latestMerge.revision">{{ shortenHash(latestMerge.revision) }}</pre>
-      <span v-if="!shouldShowRevisionEvent"
-        >at commit
-        <pre :title="latestMerge.commit">{{ shortenHash(latestMerge.commit) }}</pre></span
-      >&ensp;<wbr /><pre :title="getFormattedDate(latestMerge.timestamp, timeLocale)">{{
+      &ensp;<wbr /><pre :title="getFormattedDate(latestMerge.timestamp, timeLocale)">{{
         mergedTimeAgo
       }}</pre>
     </div>
@@ -53,8 +50,7 @@ setInterval(() => {
         getIdentityAliasOrId(latestRevision.author)
       }}</pre>
       with revision
-      <pre :title="latestRevision.id">{{ shortenHash(latestRevision.id) }}</pre> at commit
-      <pre :title="latestRevision.oid">{{ shortenHash(latestRevision.oid) }}</pre
+      <pre :title="latestRevision.id">{{ shortenHash(latestRevision.id) }}</pre
       >&ensp;<wbr /><pre :title="getFormattedDate(latestRevision.timestamp, timeLocale)">{{
         updatedTimeAgo
       }}</pre>