Closed (won't fix)
Project:
Follow
Version:
6.x-1.3
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
21 Aug 2009 at 10:43 UTC
Updated:
13 May 2010 at 21:40 UTC
A small issue:
function _follow_block_subject($uid = 0) {
return follow_link_title($uid) .':';
}
Why is the " : " here? If someone would want a " : ", he could set it in the t() in
return t('Follow !name on', array('!name' => theme('username', $account, array('plain' => TRUE))));
Comments
Comment #1
q0rban commentedI agree, this is a bit messy, however you could fix it pretty easily if you wanted by calling hook_preprocess_block() and changing the title there. So there's your quick fix.
Two thoughts I had:
1.) Have the colon added in the CSS, therefore easy to override..
2.) Make a theme function for the block title
I'm kind of leaning towards the former to avoid yet another theme function, except that IE's support for adding content via CSS is non-existant AFAIK
This isn't really a bug report though, as the code is functioning as designed, so I'm switching to a feature request.
Comment #2
q0rban commented