List support for tr.nolink class

The `nolink` CSS class can now be applied to table rows elements (TR) in addition to table data elements (TD) to disable the click event
ingedman · Jan 6, 2020 · 4a12951 · 4a12951
1 parent 06da92e
commit 4a12951
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 14 deletions.
diff --git a/modules/system/assets/ui/js/list.rowlink.js b/modules/system/assets/ui/js/list.rowlink.js
@@ -62,11 +62,11 @@
                 }
             }
 
-            $(this).find('td').not('.' + options.excludeClass).click(function(e) {
+            $(this).not('.' + options.excludeClass).find('td').not('.' + options.excludeClass).click(function(e) {
                 handleClick(e)
             })
 
-            $(this).on('keypress', function(e) {
+            $(this).not('.' + options.excludeClass).on('keypress', function(e) {
                 if (e.key === '(Space character)' || e.key === 'Spacebar' || e.key === ' ') {
                     handleClick(e)
                     return false

diff --git a/modules/system/assets/ui/less/list.less b/modules/system/assets/ui/less/list.less
@@ -153,7 +153,7 @@ table.table.data {
             }
         }
 
-        tr.rowlink:hover td, tr:not(.no-data).selected td {
+        tr.rowlink:not(.nolink):hover td, tr:not(.no-data).selected td {
             background: @color-list-hover-bg !important;
             color: @highlight-hover-text;
 
@@ -162,7 +162,7 @@ table.table.data {
             }
         }
 
-        tr.rowlink:active td {
+        tr.rowlink:not(.nolink):active td {
             background: @color-list-active-bg !important;
             color: @color-list-active-text;
         }

diff --git a/modules/system/assets/ui/less/list.rowlink.less b/modules/system/assets/ui/less/list.rowlink.less
@@ -2,7 +2,7 @@
 // CSS for rowlink plugin
 // ----------------------
 
-tr.rowlink td {
+tr.rowlink:not(.nolink) td {
     cursor: pointer;
     &.nolink {
         cursor: auto;

diff --git a/modules/system/assets/ui/storm-min.js b/modules/system/assets/ui/storm-min.js
diff --git a/modules/system/assets/ui/storm.css b/modules/system/assets/ui/storm.css
@@ -4288,8 +4288,8 @@ body.slim-container .form-buttons {padding:0 20px 20px}
 [data-control=toolbar] .select2-container .select2-selection__rendered {line-height:17px}
 [data-control=toolbar] .select2-container .select2-selection--single {height:36px}
 [data-control=toolbar] select.form-control.custom-select {display:none}
-tr.rowlink td {cursor:pointer}
-tr.rowlink td.nolink {cursor:auto}
+tr.rowlink:not(.nolink) td {cursor:pointer}
+tr.rowlink:not(.nolink) td.nolink {cursor:auto}
 .table tbody tr.rowlink:hover td {background-color:#f5f5f5}
 a.rowlink {color:inherit;font:inherit;text-decoration:inherit}
 table {max-width:100%;background-color:transparent}
@@ -4472,15 +4472,15 @@ table.table.data tbody tr:last-child td {border-bottom-width:0}
 table.table.data tbody tr td:first-child {border-left:3px solid transparent;padding-left:22px}
 table.table.data tbody tr.active td {color:#000}
 table.table.data tbody tr.active td:first-child {border-left:3px solid #f93}
-table.table.data tbody tr.rowlink:hover td,
+table.table.data tbody tr.rowlink:not(.nolink):hover td,
 table.table.data tbody tr:not(.no-data).selected td {background:#4ea5e0 !important;color:#fff}
-table.table.data tbody tr.rowlink:hover td a:not(.btn),
+table.table.data tbody tr.rowlink:not(.nolink):hover td a:not(.btn),
 table.table.data tbody tr:not(.no-data).selected td a:not(.btn),
-table.table.data tbody tr.rowlink:hover td span,
+table.table.data tbody tr.rowlink:not(.nolink):hover td span,
 table.table.data tbody tr:not(.no-data).selected td span,
-table.table.data tbody tr.rowlink:hover td i[class^="icon-"],
+table.table.data tbody tr.rowlink:not(.nolink):hover td i[class^="icon-"],
 table.table.data tbody tr:not(.no-data).selected td i[class^="icon-"] {color:#fff}
-table.table.data tbody tr.rowlink:active td {background:#3498db !important;color:#fff}
+table.table.data tbody tr.rowlink:not(.nolink):active td {background:#3498db !important;color:#fff}
 table.table.data tbody tr.hidden td,
 table.table.data tbody tr.hidden th,
 table.table.data tbody tr.hidden td a,