From a559dc0be739d6ff18749c17135ce3c403575171 Mon Sep 17 00:00:00 2001
From: Frederic Guillot <fred@kanboard.net>
Date: Tue, 14 Feb 2017 22:09:29 -0500
Subject: Use contextual menu instead of action column in users management

---
 app/Template/group/index.php | 26 +++++++++++---------------
 app/Template/group/users.php | 18 ++++++++++++------
 2 files changed, 23 insertions(+), 21 deletions(-)

(limited to 'app/Template/group')

diff --git a/app/Template/group/index.php b/app/Template/group/index.php
index fe3f398e..e4856a70 100644
--- a/app/Template/group/index.php
+++ b/app/Template/group/index.php
@@ -11,31 +11,27 @@
         <table class="table-fixed table-scrolling">
             <tr>
                 <th class="column-5"><?= $paginator->order(t('Id'), 'id') ?></th>
-                <th class="column-20"><?= $paginator->order(t('External Id'), 'external_id') ?></th>
                 <th><?= $paginator->order(t('Name'), 'name') ?></th>
-                <th class="column-5"><?= t('Actions') ?></th>
+                <th class="column-30"><?= $paginator->order(t('External Id'), 'external_id') ?></th>
             </tr>
             <?php foreach ($paginator->getCollection() as $group): ?>
             <tr>
                 <td>
-                    #<?= $group['id'] ?>
-                </td>
-                <td>
-                    <?= $this->text->e($group['external_id']) ?>
+                    <div class="dropdown">
+                        <a href="#" class="dropdown-menu dropdown-menu-link-icon">#<?= $group['id'] ?> <i class="fa fa-caret-down"></i></a>
+                        <ul>
+                            <li><?= $this->modal->medium('plus', t('Add group member'), 'GroupListController', 'associate', array('group_id' => $group['id'])) ?></li>
+                            <li><?= $this->url->icon('users', t('Members'), 'GroupListController', 'users', array('group_id' => $group['id'])) ?></li>
+                            <li><?= $this->modal->medium('edit', t('Edit'), 'GroupModificationController', 'show', array('group_id' => $group['id'])) ?></li>
+                            <li><?= $this->modal->confirm('trash-o', t('Remove'), 'GroupListController', 'confirm', array('group_id' => $group['id'])) ?></li>
+                        </ul>
+                    </div>
                 </td>
                 <td>
                     <?= $this->url->link($this->text->e($group['name']), 'GroupListController', 'users', array('group_id' => $group['id'])) ?>
                 </td>
                 <td>
-                    <div class="dropdown">
-                    <a href="#" class="dropdown-menu dropdown-menu-link-icon"><i class="fa fa-cog fa-fw"></i><i class="fa fa-caret-down"></i></a>
-                    <ul>
-                        <li><?= $this->modal->medium('plus', t('Add group member'), 'GroupListController', 'associate', array('group_id' => $group['id'])) ?></li>
-                        <li><?= $this->url->icon('users', t('Members'), 'GroupListController', 'users', array('group_id' => $group['id'])) ?></li>
-                        <li><?= $this->modal->medium('edit', t('Edit'), 'GroupModificationController', 'show', array('group_id' => $group['id'])) ?></li>
-                        <li><?= $this->modal->confirm('trash-o', t('Remove'), 'GroupListController', 'confirm', array('group_id' => $group['id'])) ?></li>
-                    </ul>
-                    </div>
+                    <?= $this->text->e($group['external_id']) ?>
                 </td>
             </tr>
             <?php endforeach ?>
diff --git a/app/Template/group/users.php b/app/Template/group/users.php
index ef179674..268f28de 100644
--- a/app/Template/group/users.php
+++ b/app/Template/group/users.php
@@ -10,16 +10,25 @@
     <?php else: ?>
         <table class="table-striped table-scrolling">
             <tr>
-                <th><?= $paginator->order(t('Id'), 'id') ?></th>
+                <th class="column-5"><?= $paginator->order(t('Id'), 'id') ?></th>
                 <th><?= $paginator->order(t('Username'), 'username') ?></th>
                 <th><?= $paginator->order(t('Name'), 'name') ?></th>
                 <th><?= $paginator->order(t('Email'), 'email') ?></th>
-                <th><?= t('Actions') ?></th>
             </tr>
             <?php foreach ($paginator->getCollection() as $user): ?>
             <tr>
                 <td>
-                    <?= $this->url->link('#'.$user['id'], 'UserViewController', 'show', array('user_id' => $user['id'])) ?>
+                    <div class="dropdown">
+                        <a href="#" class="dropdown-menu dropdown-menu-link-icon"><?= '#'.$user['id'] ?> <i class="fa fa-caret-down"></i></a>
+                        <ul>
+                            <li>
+                                <?= $this->url->icon('user', t('View profile'), 'UserViewController', 'show', array('user_id' => $user['id'])) ?>
+                            </li>
+                            <li>
+                                <?= $this->modal->confirm('trash-o', t('Remove this user from group'), 'GroupListController', 'dissociate', array('group_id' => $group['id'], 'user_id' => $user['id'])) ?>
+                            </li>
+                        </ul>
+                    </div>
                 </td>
                 <td>
                     <?= $this->url->link($this->text->e($user['username']), 'UserViewController', 'show', array('user_id' => $user['id'])) ?>
@@ -30,9 +39,6 @@
                 <td>
                     <a href="mailto:<?= $this->text->e($user['email']) ?>"><?= $this->text->e($user['email']) ?></a>
                 </td>
-                <td>
-                    <?= $this->modal->confirm('trash-o', t('Remove this user'), 'GroupListController', 'dissociate', array('group_id' => $group['id'], 'user_id' => $user['id'])) ?>
-                </td>
             </tr>
             <?php endforeach ?>
         </table>
-- 
cgit v1.2.3