From f816273a3ea18146a409f5b6ace5e6be445f3155 Mon Sep 17 00:00:00 2001
From: Andrew Davis <andrew@affinitysoftware.net>
Date: Wed, 26 May 2010 02:05:58 +0000
Subject: [PATCH] blocks MDL-22564 removed some commented out old code and set
 online users popup height and width to what they apparently should be

---
 blocks/online_users/block_online_users.php | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/blocks/online_users/block_online_users.php b/blocks/online_users/block_online_users.php
index e0aabba7fa2..9cc727f08d5 100644
--- a/blocks/online_users/block_online_users.php
+++ b/blocks/online_users/block_online_users.php
@@ -154,11 +154,10 @@ class block_online_users extends block_base {
                     $this->content->text .= '<a href="'.$CFG->wwwroot.'/user/view.php?id='.$user->id.'&amp;course='.$this->page->course->id.'" title="'.$timeago.'">'.$user->fullname.'</a></div>';
                 }
                 if ($canshowicon and ($USER->id != $user->id) and  $user->username != 'guest') {  // Only when logged in and messaging active etc
-                    //<a title="'.get_string('messageselectadd').'" href="'.$CFG->wwwroot.'/message/discussion.php?id='.$user->id.'" onclick="this.target=\'message_'.$user->id.'\';return openpopup(\'/message/discussion.php?id='.$user->id.'\', \'message_'.$user->id.'\', \'menubar=0,location=0,scrollbars,status,resizable,width=400,height=500\', 0);">
                     $link = '/message/discussion.php?id='.$user->id;
                     $anchortagcontents = '<img class="iconsmall" src="'.$OUTPUT->pix_url('t/message') . '" alt="'. get_string('messageselectadd') .'" />';
 
-                    $action = new popup_action('click', $link, 'message_'.$user->id);
+                    $action = new popup_action('click', $link, 'message_'.$user->id, array('width'=>400,'height'=>500));
                     $anchortag = $OUTPUT->action_link($link, $anchortagcontents, $action, array('title'=>get_string('messageselectadd')));
 
                     $this->content->text .= '<div class="message">'.$anchortag.'</div>';