From 660c28e2f8145b32cca66713b403bbcf8be418c1 Mon Sep 17 00:00:00 2001
From: Mike Koch <mkoch227@gmail.com>
Date: Mon, 2 Jan 2017 16:03:20 -0500
Subject: [PATCH] Move users online to top navbar

---
 css/mods-for-hesk-new.css  |  4 +++
 inc/footer.inc.php         |  6 ----
 inc/show_admin_nav.inc.php |  9 +++++-
 inc/users_online.inc.php   | 64 ++++++++++++++++++++++----------------
 4 files changed, 49 insertions(+), 34 deletions(-)

diff --git a/css/mods-for-hesk-new.css b/css/mods-for-hesk-new.css
index 72c974fa..fdf5f5b5 100644
--- a/css/mods-for-hesk-new.css
+++ b/css/mods-for-hesk-new.css
@@ -265,4 +265,8 @@ div.ticket-info {
 .box-header h1.box-title {
     width: 100%;
     cursor: pointer;
+}
+
+.black {
+    color: black;
 }
\ No newline at end of file
diff --git a/inc/footer.inc.php b/inc/footer.inc.php
index 9d50c8a9..e7c6c02d 100644
--- a/inc/footer.inc.php
+++ b/inc/footer.inc.php
@@ -43,12 +43,6 @@ if (defined('AUTOFOCUS'))
 <?php
 }
 
-// Users online
-if (defined('SHOW_ONLINE'))
-{
-	hesk_printOnline();
-}
-
 // The closing div here is to close the content area on each page. Annoying, but necessary.
 if (defined('ADMIN_PAGE')) {
 	echo '
diff --git a/inc/show_admin_nav.inc.php b/inc/show_admin_nav.inc.php
index d1598dc9..d387cf36 100644
--- a/inc/show_admin_nav.inc.php
+++ b/inc/show_admin_nav.inc.php
@@ -188,7 +188,14 @@ $mails = mfh_get_mail_headers_for_dropdown($_SESSION['id'], $hesk_settings, $hes
                                 </li>
                             </ul>
                         </li>
-                    <?php endif; ?>
+                    <?php
+                    endif;
+
+                    // Users online
+                    if (defined('SHOW_ONLINE')) {
+                        hesk_printOnline();
+                    }
+                    ?>
                     <li class="dropdown messages-menu">
                         <a href="#" class="dropdown-toggle" data-toggle="dropdown">
                             <i class="fa fa-envelope-o"></i>
diff --git a/inc/users_online.inc.php b/inc/users_online.inc.php
index 350d5237..1e6fd75c 100755
--- a/inc/users_online.inc.php
+++ b/inc/users_online.inc.php
@@ -33,34 +33,44 @@ function hesk_initOnline($user_id)
 } // END hesk_initOnline()
 
 
-function hesk_printOnline()
-{
+function hesk_printOnline() {
 	global $hesk_settings, $hesklang;
-
-	echo '
-    &nbsp;<br />&nbsp;
-	<div class="online">
-
-	<table border="0">
-	<tr>
-	<td valign="top"><img src="../img/online_on.png" width="16" height="16" alt="'.$hesklang['onlinep'].'" title="'.$hesklang['onlinep'].'" style="vertical-align:text-bottom" /></td>
-	<td>
-	';
-	$i = '';
-	foreach ($hesk_settings['users_online'] as $tmp)
-	{
-		$i .= '<span class="online" ' . ($tmp['isadmin'] ? 'style="font-style:italic;"' : '') . '>';
-		$i .= ($tmp['id'] == $_SESSION['id']) ? $tmp['name'] : '<a href="mail.php?a=new&id='.$tmp['id'].'">' . $tmp['name'] . '</a>';
-		$i .= '</span>, ';
-	}
-	echo substr($i,0,-2);
-	echo '
-	</td>
-	</tr>
-	</table>
-
-	</div>';
-
+    ?>
+    <li class="dropdown messages-menu">
+        <a href="#" class="dropdown-toggle" data-toggle="dropdown">
+            <i class="fa fa-users"></i>
+            <span class="label label-danger"><?php echo count($hesk_settings['users_online']); ?></span>
+        </a>
+        <ul class="dropdown-menu">
+            <li class="header"><?php echo $hesklang['onlinep']; ?></li>
+            <li>
+                <ul class="menu">
+                    <?php foreach ($hesk_settings['users_online'] as $tmp): ?>
+                        <li>
+                            <?php $link = ($tmp['id'] == $_SESSION['id']) ? '' : ' href="mail.php?a=new&id='.$tmp['id'].'"'; ?>
+                            <a<?php echo $link; ?>>
+                                <div class="pull-left">
+                                    <i class="fa fa-user fa-2x black"></i>
+                                </div>
+                                <h4><?php echo $tmp['name']; ?></h4>
+                                <?php
+                                if ($tmp['isadmin']): ?>
+                                    <p>
+                                        <span class="label label-default">
+                                            <?php echo $hesklang['administrator']; ?>
+                                        </span>
+                                    </p>
+                                <?php elseif ($tmp['id'] == $_SESSION['id']): ?>
+                                    <!--<span class="badge badge-default">You</span>-->
+                                <?php endif; ?>
+                            </a>
+                        </li>
+                    <?php endforeach; ?>
+                </ul>
+            </li>
+        </ul>
+    </li>
+<?php
 } // END hesk_printOnline()