Add Avatar To Memberlist

Since mostly everybody agree with this here: http://www.elkarte.net/community/index.php?topic=2398.0, this is the PR to add avatar to memberlist. Signed-off: Ahmad Rasyid Ismail "ahrasis" ahrasis@gmail.com.
elkarte · Feb 3, 2015 · 419f9bf · 419f9bf
1 parent 1e0ce9d
commit 419f9bf
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 8 deletions.
diff --git a/sources/controllers/Memberlist.controller.php b/sources/controllers/Memberlist.controller.php
@@ -69,13 +69,9 @@ public function action_index()
 
 		// Set up the standard columns...
 		$context['columns'] = array(
-			'online' => array(
-				'label' => $txt['status'],
-				'class' => 'status',
-				'sort' => array(
-					'down' => allowedTo('moderate_forum') ? 'IFNULL(lo.log_time, 1) ASC, real_name ASC' : 'CASE WHEN mem.show_online THEN IFNULL(lo.log_time, 1) ELSE 1 END ASC, real_name ASC',
-					'up' => allowedTo('moderate_forum') ? 'IFNULL(lo.log_time, 1) DESC, real_name DESC' : 'CASE WHEN mem.show_online THEN IFNULL(lo.log_time, 1) ELSE 1 END DESC, real_name DESC'
-				),
+			'avatar' => array(
+				'label' => '',
+				'class' => '',
 			),
 			'real_name' => array(
 				'label' => $txt['username'],
@@ -85,6 +81,14 @@ public function action_index()
 					'up' => 'mem.real_name ASC'
 				),
 			),
+			'online' => array(
+				'label' => $txt['status'],
+				'class' => 'status',
+				'sort' => array(
+					'down' => allowedTo('moderate_forum') ? 'IFNULL(lo.log_time, 1) ASC, real_name ASC' : 'CASE WHEN mem.show_online THEN IFNULL(lo.log_time, 1) ELSE 1 END ASC, real_name ASC',
+					'up' => allowedTo('moderate_forum') ? 'IFNULL(lo.log_time, 1) DESC, real_name DESC' : 'CASE WHEN mem.show_online THEN IFNULL(lo.log_time, 1) ELSE 1 END DESC, real_name DESC'
+				),
+			),
 			'email_address' => array(
 				'label' => $txt['email'],
 				'class' => 'email',

diff --git a/sources/subs/Memberlist.subs.php b/sources/subs/Memberlist.subs.php
@@ -344,6 +344,7 @@ function printMemberListRows($request)
 		$context['members'][$member]['post_percent'] = round(($context['members'][$member]['real_posts'] * 100) / $most_posts);
 		$context['members'][$member]['registered_date'] = strftime('%Y-%m-%d', $context['members'][$member]['registered_timestamp']);
 		$context['members'][$member]['real_name'] = $context['members'][$member]['link'];
+		$context['members'][$member]['avatar'] = '<a href="' . $context['members'][$member]['href'] . '" target="_blank" class="new_win">' . $context['members'][$member]['avatar']['image'] . '</a>';
 		$context['members'][$member]['email_address'] = $context['members'][$member]['email'];
 		$context['members'][$member]['website_url'] = $context['members'][$member]['website']['url'] != '' ? '<a href="' . $context['members'][$member]['website']['url'] . '" target="_blank" class="new_win"><img src="' . $settings['images_url'] . '/profile/www.png" alt="' . $context['members'][$member]['website']['title'] . '" title="' . $context['members'][$member]['website']['title'] . '" /></a>' : '';
 		$context['members'][$member]['id_group'] = empty($context['members'][$member]['group']) ? $context['members'][$member]['post_group'] : $context['members'][$member]['group'];

diff --git a/themes/default/css/index.css b/themes/default/css/index.css
@@ -3791,7 +3791,12 @@ dl.no_members {
 	padding: 12px 0;
 }
 #memberlist .status {
-	width: 60px;
+	width: 20px;
+}
+#memberlist .avatar.avatarresize {
+	width: 2.4em;
+	heigth: 2.4em;
+	margin: -4px -1.5em -8px 0;
 }
 #memberlist .selected {
 	width: auto;