Add clickability to all data rows when a default action is obvious (CO-1949)

app/templates/ApiUsers/columns.inc

@@ -35,7 +35,8 @@ if(!empty($vv_cur_mg)) {
 
 $indexColumns = [
   'username' => [
-    'type' => 'link'
+    'type' => 'link',
+    'cssClass' => 'row-link'
   ]
 ];
 

app/templates/AttributeGroups/columns.inc

@@ -27,6 +27,7 @@
 
 $indexColumns = [
   'name' => [
-    'type' => 'link'
+    'type' => 'link',
+    'cssClass' => 'row-link'
   ]
 ];

app/templates/AttributeMappings/columns.inc

@@ -29,7 +29,8 @@
 $indexColumns = [
   'query' => [
     'type' => 'link',
-    'sortable' => true
+    'sortable' => true,
+    'cssClass' => 'row-link'
   ],
   'value' => [
     'type' => 'echo',

app/templates/AttributeMaps/columns.inc

@@ -27,7 +27,8 @@
 
 $indexColumns = [
   'name' => [
-    'type' => 'link'
+    'type' => 'link',
+    'cssClass' => 'row-link'
   ]
 ];
 

app/templates/Attributes/columns.inc

@@ -27,7 +27,8 @@
 
 $indexColumns = [
   'name' => [
-    'type' => 'link'
+    'type' => 'link',
+    'cssClass' => 'row-link'
   ],
   'attribute_group_id' => [
     'type' => 'fk'

app/templates/Matchgrids/columns.inc

@@ -27,7 +27,8 @@
 
 $indexColumns = [
   'table_name' => [
-    'type' => 'link'
+    'type' => 'link',
+    'cssClass' => 'row-link'
   ],
   'status' => [
     'type' => 'enum',

app/templates/Matchgrids/pending.php

@@ -44,13 +44,12 @@
   </tr>
   <?php foreach($vv_pending as $p): ?>
   <tr class="linked-row spin">
-    <td>
+    <td class="row-link">
       <?= $this->Html->link(
         $p['sorid'],
         ['action' => 'reconcile',
           $vv_matchgrid_id,
-          '?' => [ 'rowid' => $p['id'] ] ],
-        ['class' => 'row-link']
+          '?' => [ 'rowid' => $p['id'] ] ]
       ); ?>
     </td>
     <td>

app/templates/Permissions/columns.inc

@@ -27,7 +27,8 @@
 
 $indexColumns = [
   'username' => [
-    'type' => 'link'
+    'type' => 'link',
+    'cssClass' => 'row-link'
   ],
   'matchgrid_id' => [ 'type' => 'fk' ],
   'permission' => [

app/templates/RuleAttributes/columns.inc

@@ -30,7 +30,8 @@ $indexColumns = [
     'type' => 'link',
     'model' => 'attribute',
     'field' => 'name',
-    'sortable' => 'Attributes.name'
+    'sortable' => 'Attributes.name',
+    'cssClass' => 'row-link'
   ],
   'crosscheck_attribute_id' => [
     'type' => 'fk',

app/templates/Rules/columns.inc

@@ -27,7 +27,8 @@
 
 $indexColumns = [
   'name' => [
-    'type' => 'link'
+    'type' => 'link',
+    'cssClass' => 'row-link'
   ],
   'confidence_mode' => [
     'type' => 'enum',

app/templates/Standard/index.php

@@ -143,7 +143,7 @@ function _column_key($modelsName, $c, $tz=null) {
   <table id="<?= $vv_tablename . '-table'; ?>">
     <tr>
       <?php foreach($indexColumns as $col => $cfg): ?>
-      <th><?php
+      <th<?= !empty($cfg['cssClass']) ? ' class="' . $cfg['cssClass'] . '"' : ''; ?>><?php
         $label = !empty($cfg['label']) ? $cfg['label'] : _column_key($modelsName, $col, $vv_tz);
 
         if(isset($cfg['sortable']) && $cfg['sortable']) {
@@ -171,7 +171,7 @@ function _column_key($modelsName, $c, $tz=null) {
     ?>
     <tr>
       <?php foreach($indexColumns as $col => $cfg): ?>
-      <td>
+      <td<?= !empty($cfg['cssClass']) ? ' class="' . $cfg['cssClass'] . '"' : ''; ?>>
         <?php
           switch($cfg['type']) {
             case 'boolean':

app/templates/SystemsOfRecord/columns.inc

@@ -27,7 +27,8 @@
 
 $indexColumns = [
   'label' => [
-    'type' => 'link'
+    'type' => 'link',
+    'cssClass' => 'row-link'
   ],
   'resolution_mode' => [
     'type' => 'enum',

app/templates/element/javascript.php

@@ -175,11 +175,24 @@
 
     // Enable Bootstrap Popovers. Unless needed elsewhere, constrain this to #content
     $('#content [data-toggle="popover"]').popover();
-    
-    // Generic row click handling
-    $('.linked-row').click(function (e) {
-       location.href = $(this).find('a.row-link').attr('href');
+
+    // Generic row click handling for div-based rows
+    $('div.linked-row').click(function (e) {
+      location.href = $(this).find('a.row-link').attr('href');
     });
+
+    // Generic row click handling for table rows
+    $('td.row-link').each(function (e) {
+      url = $(this).find('a').attr('href');
+      if(url != undefined && url != '') {
+        $(this).closest('tr').addClass('linked-row').attr('data-target',url).click(function (e) {
+          location.href = $(this).attr('data-target');
+        }).find('a').on('click',function(e){
+          // don't propagate on other links to avoid redirecting dialog boxes
+          e.stopPropagation();
+        });  
+      }
+    });    
 
     // Add loading animation when a form is submitted, when any item with a "spin" class is clicked, 
     // or on any button or anchor tag lacking the .nospin class.
@@ -203,4 +216,4 @@
 
 </script>
 
-<?= $this->Flash->render() ?>
+<?= $this->Flash->render() ?>

app/webroot/css/co-base.css

@@ -1161,7 +1161,7 @@ td.indented {
 }
 .linked-row:hover,
 .linked-row:hover td {
-  background-color: var(--cmg-color-lightgray-002) !important;
+  background-color: var(--cmg-color-lightgray-004) !important;
 }
 .menuitembutton {
   width: 250px;