#57 fix: only show delete button if bookmark can be deleted

qownnotes · Feb 10, 2024 · e13e200 · e13e200
1 parent 3c1c2d4
commit e13e200
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # QOwnNotes Web Companion Changelog
 
+## 2024.2.4
+- now only a delete button is shown in the bookmarks list if the bookmark
+  can be deleted (for [#57](https://github.com/qownnotes/web-companion/issues/57))
+
 ## 2024.2.3
 - the missing delete code was added (for [#57](https://github.com/qownnotes/web-companion/issues/57))
 - the minimum height of the popup was raised to 300px to allow

diff --git a/src/pages/PopupPage.vue b/src/pages/PopupPage.vue
@@ -147,7 +147,7 @@
                   </div>
                 </q-td>
                 <q-td key="toolbar" :props="props">
-                  <q-btn size="xs" round color="secondary" icon="delete" @click="deleteBookmark(props.row.markdown)">
+                  <q-btn v-if="props.row.markdown" size="xs" round color="secondary" icon="delete" @click="deleteBookmark(props.row.markdown)">
                     <q-tooltip class="bg-accent">{{ getLocale('DeleteBookmark') }}</q-tooltip>
                   </q-btn>
                 </q-td>