From ac666df4ceea92a12fa648e885094a3a1b65a730 Mon Sep 17 00:00:00 2001
From: shamoon <4887959+shamoon@users.noreply.github.com>
Date: Sun, 12 May 2024 19:57:08 -0700
Subject: [PATCH] Fix: only use pointer for sortable table headings, fix title
 width

---
 .../document-list.component.html              | 20 +++++++++----------
 .../document-list.component.scss              |  2 +-
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/src-ui/src/app/components/document-list/document-list.component.html b/src-ui/src/app/components/document-list/document-list.component.html
index 3030909c4..b0a5b92e6 100644
--- a/src-ui/src/app/components/document-list/document-list.component.html
+++ b/src-ui/src/app/components/document-list/document-list.component.html
@@ -161,7 +161,7 @@
         <thead>
           <th></th>
           @if (activeDisplayFields.includes(DisplayField.ASN)) {
-            <th class="d-none d-lg-table-cell"
+            <th class="d-none d-lg-table-cell cursor-pointer"
               pngxSortable="archive_serial_number"
               title="Sort by ASN" i18n-title
               [currentSortField]="list.sortField"
@@ -170,7 +170,7 @@
             i18n>ASN</th>
           }
           @if (activeDisplayFields.includes(DisplayField.CORRESPONDENT) && permissionService.currentUserCan(PermissionAction.View, PermissionType.Correspondent)) {
-            <th class="d-none d-md-table-cell"
+            <th class="d-none d-md-table-cell cursor-pointer"
               pngxSortable="correspondent__name"
               title="Sort by correspondent" i18n-title
               [currentSortField]="list.sortField"
@@ -179,7 +179,7 @@
             i18n>Correspondent</th>
           }
           @if (activeDisplayFields.includes(DisplayField.TITLE)) {
-            <th
+            <th class="cursor-pointer"
               pngxSortable="title"
               title="Sort by title" i18n-title
               [currentSortField]="list.sortField"
@@ -191,7 +191,7 @@
             <th i18n>Tags</th>
           }
           @if (activeDisplayFields.includes(DisplayField.OWNER) && permissionService.currentUserCan(PermissionAction.View, PermissionType.User)) {
-            <th class="d-none d-xl-table-cell"
+            <th class="d-none d-xl-table-cell cursor-pointer"
               pngxSortable="owner"
               title="Sort by owner" i18n-title
               [currentSortField]="list.sortField"
@@ -200,7 +200,7 @@
             i18n>Owner</th>
           }
           @if (activeDisplayFields.includes(DisplayField.NOTES) && notesEnabled) {
-            <th class="d-none d-xl-table-cell"
+            <th class="d-none d-xl-table-cell cursor-pointer"
               pngxSortable="num_notes"
               title="Sort by notes" i18n-title
               [currentSortField]="list.sortField"
@@ -209,7 +209,7 @@
             i18n>Notes</th>
           }
           @if (activeDisplayFields.includes(DisplayField.DOCUMENT_TYPE) && permissionService.currentUserCan(PermissionAction.View, PermissionType.DocumentType)) {
-            <th class="d-none d-xl-table-cell"
+            <th class="d-none d-xl-table-cell cursor-pointer"
               pngxSortable="document_type__name"
               title="Sort by document type" i18n-title
               [currentSortField]="list.sortField"
@@ -218,7 +218,7 @@
             i18n>Document type</th>
           }
           @if (activeDisplayFields.includes(DisplayField.STORAGE_PATH) && permissionService.currentUserCan(PermissionAction.View, PermissionType.StoragePath)) {
-            <th class="d-none d-xl-table-cell"
+            <th class="d-none d-xl-table-cell cursor-pointer"
               pngxSortable="storage_path__name"
               title="Sort by storage path" i18n-title
               [currentSortField]="list.sortField"
@@ -227,7 +227,7 @@
             i18n>Storage path</th>
           }
           @if (activeDisplayFields.includes(DisplayField.CREATED)) {
-            <th
+            <th class="cursor-pointer"
               pngxSortable="created"
               title="Sort by created date" i18n-title
               [currentSortField]="list.sortField"
@@ -236,7 +236,7 @@
             i18n>Created</th>
           }
           @if (activeDisplayFields.includes(DisplayField.ADDED)) {
-            <th
+            <th class="cursor-pointer"
               pngxSortable="added"
               title="Sort by added date" i18n-title
               [currentSortField]="list.sortField"
@@ -277,7 +277,7 @@
                 </td>
               }
               @if (activeDisplayFields.includes(DisplayField.TITLE) || activeDisplayFields.includes(DisplayField.TAGS)) {
-                <td>
+                <td width="30%">
                   @if (activeDisplayFields.includes(DisplayField.TITLE)) {
                     <a routerLink="/documents/{{d.id}}" title="Edit document" i18n-title style="overflow-wrap: anywhere;">{{d.title | documentTitle}}</a>
                   }
diff --git a/src-ui/src/app/components/document-list/document-list.component.scss b/src-ui/src/app/components/document-list/document-list.component.scss
index 4959d52ef..fb7db4b01 100644
--- a/src-ui/src/app/components/document-list/document-list.component.scss
+++ b/src-ui/src/app/components/document-list/document-list.component.scss
@@ -6,7 +6,7 @@ tr {
   user-select: none;
 }
 
-th {
+.cursor-pointer {
   cursor: pointer;
 }