Smileys should be case-sensitive

po8 - March 17, 2006 - 06:43
Project:Smileys
Version:4.6.x-1.x-dev
Component:Code
Category:bug report
Priority:minor
Assigned:Unassigned
Status:won't fix
Description

Smileys such as :-D should be case sensitive: :-d should not match. This was important to me because of my "guy with glasses" smiley (B), which was confused by the module with (b).

The attached trivial patch fixes this.

AttachmentSize
smileys.module.patch987 bytes

#1

Gurpartap Singh - November 25, 2006 - 11:25
Status:needs review» needs work

I think this should be made optional. Marking needs work.

#2

Gurpartap Singh - February 20, 2007 - 20:56
Status:needs work» won't fix

No plans to implement this as it's better when importing external smileys packs which generally don't have both :-D :-d acronyms for corresponding smiley or so.

 
 

Drupal is a registered trademark of Dries Buytaert.