Hide move & delete buttons for new resources in list view [notarize]

dainst · Mar 24, 2023 · 2f8b386 · 2f8b386
1 parent b47d9fe
commit 2f8b386
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 9 deletions.
diff --git a/desktop/src/app/components/resources/list/row.component.ts b/desktop/src/app/components/resources/list/row.component.ts
@@ -80,6 +80,8 @@ export class RowComponent implements AfterViewInit, OnChanges {
         ? 'resource-' + this.document.resource.identifier
         : 'new-resource';
 
+    public isNewResource = () => !this.document.resource.id;
+
 
     ngAfterViewInit() {
 
@@ -151,7 +153,14 @@ export class RowComponent implements AfterViewInit, OnChanges {
 
     public isMoveOptionAvailable(): boolean {
 
-        return this.projectConfiguration.getHierarchyParentCategories(this.document.resource.category).length > 0;
+        return !this.isNewResource()
+            && this.projectConfiguration.getHierarchyParentCategories(this.document.resource.category).length > 0;
+    }
+
+
+    public isDeleteOptionAvailable(): boolean {
+
+        return !this.isNewResource();
     }
 
 
@@ -199,7 +208,7 @@ export class RowComponent implements AfterViewInit, OnChanges {
     private async save() {
 
         if (!this.document.resource.identifier || this.document.resource.identifier.trim() === '') {
-            if (this.document.resource.id) await this.restoreIdentifier(this.document);
+            if (!this.isNewResource()) await this.restoreIdentifier(this.document);
             return;
         }
 

diff --git a/desktop/src/app/components/resources/list/row.html b/desktop/src/app/components/resources/list/row.html
@@ -1,7 +1,7 @@
 <div id="{{makeId()}}"
      class="row-wrapper">
     <div class="label-field">
-        <div class="category-label" [class.text-info]="!document.resource.id">
+        <div class="category-label" [class.text-info]="isNewResource()">
             <i>{{getCategoryLabel()}}</i>
         </div>
 
@@ -92,31 +92,32 @@
                 (click)="editDocument()">
             <span class="mdi mdi-pencil mdi-18px"></span>
         </button>
-        <button *ngIf="isMoveOptionAvailable()"
+        <button [class.hidden-option]="!isMoveOptionAvailable()"
                 class="btn btn-small btn-link float-right list-move-button"
                 ngbTooltip="Verschieben"
                 i18n-ngbTooltip="@@resources.list.moveButton.tooltip"
                 placement="bottom"
                 container="body"
                 triggers="manual"
                 #moveTooltip="ngbTooltip"
-                (mouseover)="moveTooltip.open()"
+                (mouseover)="isMoveOptionAvailable() && moveTooltip.open()"
                 (mouseleave)="moveTooltip.close()"
                 (mousedown)="moveTooltip.close()"
-                (click)="moveDocument()">
+                (click)="isMoveOptionAvailable() && moveDocument()">
             <span class="mdi mdi-file-tree mdi-18px"></span>
         </button>
-        <button class="btn btn-small btn-link float-right list-delete-button"
+        <button [class.hidden-option]="!isDeleteOptionAvailable()"
+                class="btn btn-small btn-link float-right list-delete-button"
                 ngbTooltip="Löschen"
                 i18n-ngbTooltip="@@resources.list.deleteButton.tooltip"
                 placement="bottom"
                 container="body"
                 triggers="manual"
                 #deleteTooltip="ngbTooltip"
-                (mouseover)="deleteTooltip.open()"
+                (mouseover)="isMoveOptionAvailable() && deleteTooltip.open()"
                 (mouseleave)="deleteTooltip.close()"
                 (mousedown)="deleteTooltip.close()"
-                (click)="deleteDocument()">
+                (click)="isDeleteOptionAvailable() && deleteDocument()">
             <span class="mdi mdi-delete mdi-18px"></span>
         </button>
     </div>

diff --git a/desktop/src/app/components/resources/list/row.scss b/desktop/src/app/components/resources/list/row.scss
@@ -88,4 +88,8 @@
   .list-delete-button {
     border-radius: 32px !important;
   }
+
+  .hidden-option {
+    opacity: 0;
+  }
 }