Improve Manage Matchgrid layout: make Matchgrid History a top-link, a…

…nd shorten label to "History" (CO-2527)
COmanage · Oct 7, 2022 · 535e17a · 535e17a
1 parent 98b8a87
commit 535e17a
Show file tree

Hide file tree

Showing 4 changed files with 30 additions and 12 deletions.
diff --git a/app/resources/locales/en_US/default.po b/app/resources/locales/en_US/default.po
@@ -656,6 +656,9 @@ msgstr "Filter"
 msgid "match.op.go"
 msgstr "Go"
 
+msgid "match.op.history"
+msgstr "History"
+
 msgid "match.op.last"
 msgstr "Last"
 

diff --git a/app/src/View/Helper/MenuHelper.php b/app/src/View/Helper/MenuHelper.php
@@ -79,7 +79,8 @@ public function getMenuIcon($action) {
       'Edit'          =>  'edit',
       'Duplicate'     =>  'content_copy',
       'Default'       =>  'link',  // default icon for arbitrary menu items
-      'Delete'        =>  'delete'
+      'Delete'        =>  'delete',
+      'History'        =>  'history'
     );
 
     return $icon[$action];

diff --git a/app/templates/MatchgridRecords/columns.inc b/app/templates/MatchgridRecords/columns.inc
@@ -82,6 +82,7 @@ $indexActions = [
                           'sorid'         => $e->sorid
                         ];
                       },
-    'icon' => 'history'
+    'icon'        => 'history',
+    'label'       =>  __('match.op.history')
   ]
 ];
diff --git a/app/templates/Matchgrids/manage.php b/app/templates/Matchgrids/manage.php
@@ -31,6 +31,29 @@
   <div class="pageTitle">
     <h1><?= $vv_title; ?></h1>
   </div>
+
+  <?php 
+    // topLinks menu dropdown / button listing
+    $action_args = array();
+    $action_args['vv_attr_mdl'] = "IndexTopLinks";    
+    $action_args['vv_actions'][] = array(
+      'order' => $this->Menu->getMenuOrder('Add'),
+      'icon' => $this->Menu->getMenuIcon('History'),
+      'url' => $this->Url->build([
+        'controller' => 'MatchgridHistoryRecords',
+        'action' => 'index',
+        '?' => ['matchgrid_id' => $vv_cur_mg->id]
+      ]),
+      'label' =>  __('match.op.history')
+    );
+
+    // Output the topLinks
+    if(!empty($action_args['vv_actions'])) {
+      print '<div id="topLinks" class="field-actions">';
+      print $this->element('menuAction', $action_args);
+      print '</div>';
+    }
+  ?>  
 </div>
 
 <section class="inner-content">
@@ -61,15 +84,5 @@
       ?>
       <div class="call-to-action-text"><?= __('match.in.matchgrid.reconcile'); ?></div>
     </div>
-    <div class="call-to-action">
-      <?= $this->Html->link('<em class="material-icons" aria-hidden="true">list</em>' . __('match.op.display.history'),
-                            ['controller' => 'MatchgridHistoryRecords',
-                              'action' => 'index',
-                              '?' => ['matchgrid_id' => $vv_cur_mg->id]
-                            ],
-                            ['escape' => false, 'class' => 'btn btn-default']);
-      ?>
-      <div class="call-to-action-text"><?= __('match.in.matchgrid.history'); ?></div>
-    </div>
   </div>
 </section>