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

q0rban’s picture

Category: bug » feature

I 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.

q0rban’s picture

Status: Active » Closed (won't fix)