Could you make a small change to the Author Pane integration, please?

I need the link version to be this:

$variables['privatemsg_link'] = l(t('Send PM'). '<span></span>', $url, array('attributes' => array('class' => 'author-privatemsg-icon'), 'html' => TRUE));

The extra span and class lets it work with the CSS image replacement technique I'm now using.

Thanks,

Michelle

CommentFileSizeAuthor
#1 privatemsg_authorpane.patch939 bytesnaheemsays

Comments

naheemsays’s picture

naheemsays commented
Status: Active » Needs review
StatusFileSize
new privatemsg_authorpane.patch939 bytes

Just turning the above into a patch.

naheemsays’s picture

naheemsays commented

A question - is the . '<span></span>' bit needed?

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

@nbz - Yes, that's what makes the image replacement possible. The other class is to allow image + text. Having both gives both options depending on whether you use a small icon along with text or a large icon that has text on the icon itself.

Michelle

naheemsays’s picture

naheemsays commented

Reading this, it seems so. (Technique #8)

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Yep, that's the exact page I used. :)

Michelle

litwol’s picture

litwol commented
Status: Needs review » Fixed

Cheers

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Thanks!

Michelle

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.