Toggle delete btn and add delete method plan

cortezaproject · Oct 16, 2023 · 17eb597 · 17eb597
1 parent 8b6e157
commit 17eb597
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/client/web/compose/src/views/Admin/Pages/Edit.vue b/client/web/compose/src/views/Admin/Pages/Edit.vue
@@ -734,7 +734,7 @@
           {{ $t('general:label.cancel') }}
         </b-button>
         <b-button
-          :disabled="selectedAttachmentID"
+          :disabled="attachments && !selectedAttachmentID"
           variant="danger"
           @click="deleteIcon"
         >
@@ -1372,10 +1372,13 @@ export default {
     },
 
     deleteIcon () {
-      if (this.selectedAttachmentID) {
-        // study altered logic
-        // pass ID or whatever BE needs to del icon
-      }
+    // this.attachments = this.attachments.filter(a => a.attachmentID !== this.selectedAttachmentID)
+      // this.$ComposeAPI.?
+      // .then((attachments) => {
+        // either api will return updated list of attachments or you'll have to refetch
+        // this.attachments = attachments
+        // this.fetchAttachments()
+      // }).catch(this.toastErrorHandler(this.$t('notification:page.iconUndeleteFailed')))
     },
   },
 }

diff --git a/locale/en/corteza-webapp-compose/notification.yaml b/locale/en/corteza-webapp-compose/notification.yaml
@@ -86,6 +86,7 @@ page:
   cloneFailed: Could not clone this page
   deleteFailed: Could not delete this page
   iconFetchFailed: Could not fetch list of icons
+  iconUndeleteFailed:  Could not delete selected icon
   loadFailed: Could not load the page tree
   noPages: No pages found
   saveFailed: Could not save this page