Hi

I'm starting with drupal and I've found really useful this module for smileys. I'd suggest a small feature, special class in the Only local images are allowed. output tag. This way we could theme the smileys. Currently I'm having image align problems, smileys seems to be a little high.

Here you can see an example:

http://img116.imageshack.us/img116/3941/emoteszd8.jpg

Best regards and thanks for this module ;)

CommentFileSizeAuthor
#3 smiley-class.patch3 KBNoRandom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

NoRandom’s picture

Modification done. Here is what I did:

You have to edit the file 'smileys.module' and edit these lines:

Line 81

Change...

... $acronyms[0] .'" class="smiley-class"/></span>';

To...

... $acronyms[0] .'" class="smiley"/></span>';

Line 105

Change...

... check_plain($smiley->description) .'" class="smiley-class" />',

To...

... check_plain($smiley->description) .'" class="smiley" />',

Line 150

Change...

...check_plain($alt) ."\" />\\2", $text);

To...

...check_plain($alt) ."\" class=\"smiley\" />\\2", $text);

Line 152

Change...

... check_plain($alt) .'" alt="'. check_plain($alt) .'" />', $text);

To...

... check_plain($alt) .'" alt="'. check_plain($alt) .'" class="smiley"/>', $text);

Line 268

Change...

... check_plain($smiley->description) .'" />',

To...

... check_plain($smiley->description) .'" class="smiley" />',

This way, every image shown by the module will have an attached smiley class and you can do whatever you want from css.

VM’s picture

This has a greater degree of making into the module if a patch is provided see: http://drupal.org/patch/create

NoRandom’s picture

FileSize
3 KB

Thanks for your answer.

Here goes the patch with the changes I explained.

Gurpartap Singh’s picture

Status: Active » Closed (duplicate)