Bug 118624 - Emoticons which contain a smaller emoticon inside them do not display
Summary: Emoticons which contain a smaller emoticon inside them do not display
Status: RESOLVED DUPLICATE of bug 174262
Alias: None
Product: kopete
Classification: Applications
Component: libkopete (show other bugs)
Version: 0.40.0
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Kopete Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-19 07:42 UTC by Seth Kinast
Modified: 2009-04-06 23:31 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Fixes the sorting of the emoticons for KDE 4.1 Revision 812010 (1.50 KB, patch)
2008-08-20 20:28 UTC, Timothy Knoll
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Seth Kinast 2005-12-19 07:42:54 UTC
Version:           0.11 (using KDE KDE 3.5.0)
Installed from:    Ubuntu Packages
OS:                Linux

Yeah, a short description was hard to come up with. Here's an example: I have two emoticons:

:r and :rolleyes:
Notice that :r is the first part of :rolleyes:

If I have "Emoticons must have spaces around them" turned on,
:r => [emot :r]
:rolleyes: => :rolleyes: // NO EMOTICON PARSING

If I have "Emoticons must have spaces around them" turned off,
:r => [emot :r]
:rolleyes: => [emot :r]olleyes // (semi-)expected behavior, but the better match should be chosen first, as per http://kopete.kde.org/emoticons/emoticonspec.html#impl-parser

> Emoticon parser
>
> There are no real recommendation how to parse emoticons. But an emoticon with a longer string should have the priority over one
> with a shorter string. (Example: if both :-() and :-( are in a theme, the picture for the first one should be used when :-() is
> encountered)
Comment 1 José Villavicencio 2006-03-12 01:01:51 UTC
I'm having the same problem here with version 0.11.91, compiled from source with kde 3.5.1
Comment 2 Olivier Goffart 2006-04-15 11:10:46 UTC
According to the code, it is taken by the order they are in the xml file.


Comment 3 Seth Kinast 2006-04-15 20:17:39 UTC
Right; that makes this a bug according to the spec (written by you ;) )
Comment 4 A. Spehr 2008-07-07 09:10:49 UTC
:-() doesn't show as an emoticon when spaces are turned off

r828851 kdenetwork
Comment 5 Timothy Knoll 2008-08-20 20:28:18 UTC
Created attachment 26951 [details]
Fixes the sorting of the emoticons for KDE 4.1 Revision 812010

In revision 812010 the emoticons are unsorted. This patch fixes that.
It probably doesn't follow KDE standards but it works.
Comment 6 Timothy Knoll 2008-10-08 16:13:40 UTC
The patch sorts, using qSort, according to size largest to smallest.  This allows the rest of the code to behave as expected.


It still applies cleanly to revision 869212
Comment 7 Matt Rogers 2009-04-06 23:31:41 UTC

*** This bug has been marked as a duplicate of bug 174262 ***