From b2cae4581a0165c1ed3157155f991eee5e8d4704 Mon Sep 17 00:00:00 2001
From: wandji20 <wbertrand10@gmail.com>
Date: Sun, 24 Mar 2024 22:33:01 +0100
Subject: [PATCH] Improve logic for showing disabled edit button for unsupprted
 wopi file previews [SCI-9901]

---
 .../content/attachments/open_locally_menu.vue      | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/app/javascript/vue/shared/content/attachments/open_locally_menu.vue b/app/javascript/vue/shared/content/attachments/open_locally_menu.vue
index ca2d879f1..c4beea35c 100644
--- a/app/javascript/vue/shared/content/attachments/open_locally_menu.vue
+++ b/app/javascript/vue/shared/content/attachments/open_locally_menu.vue
@@ -1,15 +1,15 @@
 <template>
   <div class="sn-open-locally-menu" @mouseenter="fetchLocalAppInfo">
-    <div v-if="!canOpenLocally && (attachment.attributes.wopi && attachment.attributes.urls.edit_asset)">
-      <a v-if="editWopiEnabled" :href="`${attachment.attributes.urls.edit_asset}`" target="_blank"
+    <button class="btn btn-light" v-if="!editWopiSupported" style="pointer-events: all;"
+        :title="attachment.attributes.wopi_context.title" disabled="true">
+      {{ attachment.attributes.wopi_context.button_text }}
+    </button>
+    <div v-else-if="!canOpenLocally && (attachment.attributes.wopi && attachment.attributes.urls.edit_asset)">
+      <a :href="`${attachment.attributes.urls.edit_asset}`" target="_blank"
       class="block whitespace-nowrap rounded px-3 py-2.5
               hover:!text-sn-blue hover:no-underline cursor-pointer hover:bg-sn-super-light-grey">
           {{ attachment.attributes.wopi_context.button_text }}
       </a>
-      <button class="btn btn-light" v-else style="pointer-events: all;"
-        :title="attachment.attributes.wopi_context.title" disabled="true">
-        {{ attachment.attributes.wopi_context.button_text }}
-      </button>
     </div>
     <div v-else-if="!usesWebIntegration">
         <MenuDropdown
@@ -104,7 +104,7 @@ export default {
       return this.attachment.attributes.asset_type === 'gene_sequence'
         || this.attachment.attributes.asset_type === 'marvinjs';
     },
-    editWopiEnabled() {
+    editWopiSupported() {
       return this.attachment.attributes.wopi_context.edit_supported;
     }
   },