Review Request 122639: Improve the Person applet config page

Martin Klapetek martin.klapetek at gmail.com
Thu Feb 19 14:48:00 UTC 2015


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

Review request for Telepathy.


Repository: ktp-desktop-applets


Description
-------

Turns the combobox into a grid with filter (like it used to be sometime ago). See screenshots.


Diffs
-----

  person/contents/ui/settingsGeneral.qml 25f6df0 

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


Testing
-------

Selecting a contact works, everything gets saved properly.

Problem comes when reopening the config - I'm not sure how to properly find the index of the saved person. I've tried to add another SortFilterModel and filter by the personUri role, however that takes only regexp and passing the uri seems to make it parse like regexp and so returns nothing.

Any ideas on that?


File Attachments
----------------

Before
  https://git.reviewboard.kde.org/media/uploaded/files/2015/02/19/4caf0d02-fabc-4f49-8af1-ced285fa4464__person1.png
After
  https://git.reviewboard.kde.org/media/uploaded/files/2015/02/19/395f3a27-a6d3-469c-81d3-5950d3d59c61__person2.png


Thanks,

Martin Klapetek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20150219/1adf31be/attachment.html>


More information about the KDE-Telepathy mailing list