Review Request 114275: Use a QMap as source of EmoticonTextEditItem elements

[Giuseppe Calà]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114275/
-----------------------------------------------------------

(Updated Dec. 31, 2013, 1:26 p.m.)


Status
------

This change has been marked as submitted.


Review request for Telepathy.


Repository: ktp-text-ui


Description
-------

First of all, two assumptions:
1. I know that this code should be supplied upstream to kdelibs but ktp-text-ui's code is smaller and easier, for me, to manage.
2. This patch tries to fix what in my opinion is an usability issue.

Ktp-text-ui uses a QHash from KEmoticonsTheme::emoticonsMap() to build emoticons' QListWidget. The problem is that items in a QHash are arbitrarily ordered so the resulting view is messed up. With this patch the returned QHash is instead used to build a QMap, whose elements are ordered by key, and use this one as source for EmoticonTextEditItem. The final result is a view where emoticons are ordered by their filename.


Diffs
-----

  app/emoticon-text-edit-selector.cpp 1fad99b 

Diff: https://git.reviewboard.kde.org/r/114275/diff/


Testing
-------

Tested with Google Hangout Emojis Complete theme from kde-look.org:

Before: http://wstaw.org/m/2013/12/03/emoticons-view-before.png
After: http://wstaw.org/m/2013/12/03/emoticons-view-after.png


Thanks,

Giuseppe Calà

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20131231/96ba653e/attachment.html>