diff --git a/include/class.avatar.php b/include/class.avatar.php index 93024d02cb5f7b624a6e00284141b5a9ba2aac45..ee55147fc46442652bf1826386cf6f804c949ba4 100644 --- a/include/class.avatar.php +++ b/include/class.avatar.php @@ -25,7 +25,11 @@ abstract class Avatar { abstract function getUrl($size); function getImageTag($size=null) { - return '<img class="avatar" alt="'.__('Avatar').'" src="'.$this->getUrl($size).'" />'; + $style = ($size) + ? sprintf('style="max-height:%spx"', $size) + : ''; + return "<img {$style} class=\"avatar\" alt=\"" + .__('Avatar').'" src="'.$this->getUrl($size).'" />'; } function __toString() { diff --git a/include/staff/templates/collaborators-preview.tmpl.php b/include/staff/templates/collaborators-preview.tmpl.php index e7fb4f5985d6b46472e49e9453081d325c927b44..89b734ac420d7b9e95187e22bebcf23f22c98f03 100644 --- a/include/staff/templates/collaborators-preview.tmpl.php +++ b/include/staff/templates/collaborators-preview.tmpl.php @@ -5,9 +5,11 @@ if (($users=$thread->getCollaborators())) {?> <?php foreach($users as $user) { - echo sprintf('<tr><td %s><i class="icon-%s"></i> %s <em><%s></em></td></tr>', + echo sprintf('<tr><td %s>%s%s <em class="faded"><%s></em></td></tr>', ($user->isActive()? '' : 'class="faded"'), - ($user->isActive()? 'comments' : 'comment-alt'), + (($U = $user->getUser()) && ($A = $U->getAvatar())) + ? $A->getImageTag(20) : sprintf('<i class="icon-%s"></i>', + ($user->isActive()? 'comments' : 'comment-alt')), Format::htmlchars($user->getName()), $user->getEmail()); } diff --git a/include/staff/templates/collaborators.tmpl.php b/include/staff/templates/collaborators.tmpl.php index c49b1b266edaf1118a5f058ab8cb2f67fa5dc8cb..4393ba1f2fd06acb40f6d7b1f5713809cd90f773 100644 --- a/include/staff/templates/collaborators.tmpl.php +++ b/include/staff/templates/collaborators.tmpl.php @@ -15,8 +15,10 @@ if(($users=$thread->getCollaborators())) {?> $checked = $user->isActive() ? 'checked="checked"' : ''; echo sprintf('<tr> <td> + <label class="inline checkbox"> <input type="checkbox" name="cid[]" id="c%d" value="%d" %s> - <a class="collaborator" href="#thread/%d/collaborators/%d/view">%s</a> + </label> + <a class="collaborator" href="#thread/%d/collaborators/%d/view">%s%s</a> <span class="faded"><em>%s</em></span></td> <td width="10"> <input type="hidden" name="del[]" id="d%d" value=""> @@ -28,6 +30,8 @@ if(($users=$thread->getCollaborators())) {?> $checked, $thread->getId(), $user->getId(), + (($U = $user->getUser()) && ($A = $U->getAvatar())) + ? $U->getAvatar()->getImageTag(24) : '', Format::htmlchars($user->getName()), $user->getEmail(), $user->getId(), diff --git a/include/staff/templates/ticket-preview.tmpl.php b/include/staff/templates/ticket-preview.tmpl.php index 859bf0a8957b812634a774b1db0e7e2c4477f9b8..3533c8af7d8fef1dff05e00b25580e39bcf83d53 100644 --- a/include/staff/templates/ticket-preview.tmpl.php +++ b/include/staff/templates/ticket-preview.tmpl.php @@ -125,10 +125,12 @@ echo '</div>'; // ticket preview content. if ($thread && ($collabs=$thread->getCollaborators())) {?> <?php foreach($collabs as $collab) { - echo sprintf('<tr><td %s><i class="icon-%s"></i> + echo sprintf('<tr><td %s>%s <a href="users.php?id=%d" class="no-pjax">%s</a> <em><%s></em></td></tr>', ($collab->isActive()? '' : 'class="faded"'), - ($collab->isActive()? 'comments' : 'comment-alt'), + (($U = $collab->getUser()) && ($A = $U->getAvatar())) + ? $A->getImageTag(20) : sprintf('<i class="icon-%s"></i>', + $collab->isActive() ? 'comments' : 'comment-alt'), $collab->getUserId(), $collab->getName(), $collab->getEmail()); diff --git a/scp/css/scp.css b/scp/css/scp.css index 1aa6361639c51abee2bff81811197a7d4fb31669..a497ace475c552000e7c8cae23800a0ccd315c2a 100644 --- a/scp/css/scp.css +++ b/scp/css/scp.css @@ -946,6 +946,8 @@ a#post-note:hover { } img.avatar { border-radius: inherit; + vertical-align: middle; + margin-right: 5px; } .avatar > img.avatar { width: 100%;