feat(patch-list): new r-click action "Refresh Patch Data" for patches

Signed-off-by: Konstantinos Maninakis <maninak@protonmail.com>
cytechmobile · Jul 16, 2024 · 6386bf8 · 6386bf8
1 parent c524f73
commit 6386bf8
Show file tree

Hide file tree

Showing 5 changed files with 31 additions and 10 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -49,6 +49,7 @@
 - **settings:** disallow usage of now unsupported relative paths for config `radicle.advanced.pathToRadBinary`
 - **settings:** support trailing slashes for config `radicle.advanced.pathToNodeHome`
 - **patch-list:** show all possible actions for a patch on its context menu (right click)
+- **patch-list:** add new context-menu action "Refresh Patch Data" for a patch item
 
 ### 🔥 Performance
 

diff --git a/package.json b/package.json
@@ -119,13 +119,21 @@
         "enablement": "radicle.isRadInitialized"
       },
       {
-        "command": "radicle.refreshPatches",
-        "title": "Refresh Patches View",
+        "command": "radicle.refreshAllPatches",
+        "title": "Refresh All Patch Data",
         "shortTitle": "Refresh Patches",
         "category": "Radicle",
         "icon": "$(refresh)",
         "enablement": "radicle.isRadInitialized"
       },
+      {
+        "command": "radicle.refreshOnePatch",
+        "title": "Refresh Patch Data",
+        "shortTitle": "Refresh Patch",
+        "category": "Radicle",
+        "icon": "$(refresh)",
+        "enablement": "radicle.isRadInitialized"
+      },
       {
         "command": "radicle.viewPatchDetails",
         "title": "Open Patch Details",
@@ -176,6 +184,10 @@
           "command": "radicle.viewPatchDetails",
           "when": "false"
         },
+        {
+          "command": "radicle.refreshOnePatch",
+          "when": "false"
+        },
         {
           "command": "radicle.checkoutPatch",
           "when": "false"
@@ -205,7 +217,7 @@
       ],
       "view/title": [
         {
-          "command": "radicle.refreshPatches",
+          "command": "radicle.refreshAllPatches",
           "when": "view == patches-view",
           "group": "navigation@1"
         }
@@ -219,12 +231,17 @@
         {
           "command": "radicle.viewPatchDetails",
           "when": "view == patches-view && viewItem =~ /patch/",
-          "group": "contextMenu@1"
+          "group": "contextMenu.A@1"
+        },
+        {
+          "command": "radicle.refreshOnePatch",
+          "when": "view == patches-view && viewItem =~ /patch/",
+          "group": "contextMenu.A@2"
         },
         {
           "command": "radicle.copyPatchId",
           "when": "view == patches-view && viewItem =~ /patch/",
-          "group": "contextMenu@2"
+          "group": "contextMenu.C@1"
         },
         {
           "command": "radicle.checkoutPatch",
@@ -234,7 +251,7 @@
         {
           "command": "radicle.checkoutPatch",
           "when": "view == patches-view && viewItem =~ /patch:checked-out-false/",
-          "group": "contextMenu@3"
+          "group": "contextMenu.B@1"
         },
         {
           "command": "radicle.checkoutDefaultBranch",

diff --git a/src/helpers/command.ts b/src/helpers/command.ts
@@ -101,9 +101,12 @@ export function registerAllCommands(): void {
   registerVsCodeCmd('radicle.collapsePatches', () => {
     commands.executeCommand('workbench.actions.treeView.patches-view.collapseAll')
   })
-  registerVsCodeCmd('radicle.refreshPatches', () => {
+  registerVsCodeCmd('radicle.refreshAllPatches', () => {
     usePatchStore().resetAllPatches()
   })
+  registerVsCodeCmd('radicle.refreshOnePatch', (patchId: Patch['id']) => {
+    usePatchStore().refetchPatch(patchId)
+  })
   registerVsCodeCmd('radicle.checkoutPatch', checkOutPatch)
   registerVsCodeCmd('radicle.checkoutDefaultBranch', checkOutDefaultBranch)
   registerVsCodeCmd('radicle.copyPatchId', async (patch: Partial<Patch> | undefined) => {

diff --git a/src/helpers/webview.ts b/src/helpers/webview.ts
@@ -1,4 +1,4 @@
-import { Uri, ViewColumn, type Webview, type WebviewPanel, window } from 'vscode'
+import { Uri, ViewColumn, type Webview, type WebviewPanel, commands, window } from 'vscode'
 import {
   type WebviewId,
   allWebviewIds,
@@ -261,7 +261,7 @@ async function handleMessageFromWebviewPatchDetail(
       copyToClipboardAndNotify(message.payload.textToCopy)
       break
     case 'refreshPatchData':
-      usePatchStore().refetchPatch(message.payload.patchId)
+      commands.executeCommand('radicle.refreshOnePatch', message.payload.patchId)
       break
     case 'checkOutPatchBranch':
       checkOutPatch(message.payload.patch)

diff --git a/src/webviews/src/components/PatchDetailButtons.vue b/src/webviews/src/components/PatchDetailButtons.vue
@@ -27,7 +27,7 @@ function revealPatch() {
   <vscode-button
     class="self-center"
     appearance="secondary"
-    title="Refresh All Data on This Page"
+    title="Refresh Patch Data"
     @click="refetchPatchData"
   >
     <!-- eslint-disable-next-line vue/no-deprecated-slot-attribute -->