From 9ffc41a133bcadfb2d5e68eade543bc8a367dcc5 Mon Sep 17 00:00:00 2001
From: Inga Kirschnick <inga.kirschnick@uni-bielefeld.de>
Date: Wed, 7 Jun 2023 13:49:07 +0200
Subject: [PATCH] update clickable list item

---
 app/static/js/ResourceLists/CorpusFileList.js |  6 +++---
 app/static/js/ResourceLists/CorpusList.js     |  4 ++--
 app/templates/admin/user.html.j2              | 12 +-----------
 3 files changed, 6 insertions(+), 16 deletions(-)

diff --git a/app/static/js/ResourceLists/CorpusFileList.js b/app/static/js/ResourceLists/CorpusFileList.js
index 5957a140..9997b061 100644
--- a/app/static/js/ResourceLists/CorpusFileList.js
+++ b/app/static/js/ResourceLists/CorpusFileList.js
@@ -32,7 +32,7 @@ class CorpusFileList extends ResourceList {
   get item() {
     return (values) => {
       return `
-        <tr class="list-item">
+        <tr class="list-item clickable hoverable">
           <td>
             <label class="list-action-trigger ${this.hasPermissionView ? '' : 'hide'}" data-list-action="select">
               <input class="select-checkbox" type="checkbox">
@@ -79,7 +79,7 @@ class CorpusFileList extends ResourceList {
         <thead>
           <tr>
             <th>
-              <label class="selection-action-trigger ${this.listContainerElement.dataset?.hasPermissionView == 'true' ? '' : 'hide'}" data-selection-action="select-all">
+              <label class="disable-on-click selection-action-trigger ${this.listContainerElement.dataset?.hasPermissionView == 'true' ? '' : 'hide'}" data-selection-action="select-all">
                 <input class="select-all-checkbox" type="checkbox">
                 <span class="disable-on-click"></span>
               </label>
@@ -121,7 +121,7 @@ class CorpusFileList extends ResourceList {
     if (listItemElement === null) {return;}
     let itemId = listItemElement.dataset.id;
     let listActionElement = event.target.closest('.list-action-trigger[data-list-action]');
-    let listAction = listActionElement === null ? '' : listActionElement.dataset.listAction;
+    let listAction = listActionElement === null ? 'view' : listActionElement.dataset.listAction;
     switch (listAction) {
       case 'delete': {
         let values = this.listjs.get('id', itemId)[0].values();
diff --git a/app/static/js/ResourceLists/CorpusList.js b/app/static/js/ResourceLists/CorpusList.js
index c4caa399..2e41d82e 100644
--- a/app/static/js/ResourceLists/CorpusList.js
+++ b/app/static/js/ResourceLists/CorpusList.js
@@ -63,7 +63,7 @@ class CorpusList extends ResourceList {
   get item() {
     return (values) => {
       return `
-        <tr class="list-item">
+        <tr class="list-item clickable hoverable">
           <td>
             <label class="list-action-trigger" data-list-action="select">
               <input class="select-checkbox" type="checkbox">
@@ -143,7 +143,7 @@ class CorpusList extends ResourceList {
     if (listItemElement === null) {return;}
     let itemId = listItemElement.dataset.id;
     let listActionElement = event.target.closest('.list-action-trigger[data-list-action]');
-    let listAction = listActionElement === null ? '' : listActionElement.dataset.listAction;
+    let listAction = listActionElement === null ? 'view' : listActionElement.dataset.listAction;
     switch (listAction) {
       case 'delete-request': {
         let values = this.listjs.get('id', itemId)[0].values();
diff --git a/app/templates/admin/user.html.j2 b/app/templates/admin/user.html.j2
index 1fbe4c6d..82c8723d 100644
--- a/app/templates/admin/user.html.j2
+++ b/app/templates/admin/user.html.j2
@@ -56,7 +56,7 @@
     <div class="col s12" id="user-corpora">
       <div class="card">
         <div class="card-content">
-          <div class="corpus-list no-autoinit" data-user-id="{{ user.hashid }}"></div>
+          <div class="corpus-list" data-user-id="{{ user.hashid }}"></div>
         </div>
       </div>
     </div>
@@ -107,16 +107,6 @@
 {% block scripts %}
 {{ super() }}
 <script>
-
-  let corpusList = new CorpusList(document.querySelector('.corpus-list'));
-  corpusList.add(
-    [
-      {% for corpus in corpora %}
-      {{ corpus.to_json_serializeable(backrefs=True, relationships=True)|tojson }},
-      {% endfor %}
-    ]
-  );
-  
   let userRoleChip = document.querySelector('#user-role-chip');
   let userRoleChipTooltip = M.Tooltip.init(
     userRoleChip,