Review Request: Handle favorites in JoinChatroomDialog
David Edmundson
kde at davidedmundson.co.uk
Sun Mar 11 20:34:57 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104173/#review11295
-----------------------------------------------------------
Ship it!
- David Edmundson
On March 11, 2012, 7:33 p.m., Dominik Cermak wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104173/
> -----------------------------------------------------------
>
> (Updated March 11, 2012, 7:33 p.m.)
>
>
> Review request for Telepathy.
>
>
> Description
> -------
>
> Now you can maintain a list of your favorite chatrooms.
> These are saved in the general ktelepathyrc to use them for autoconnection (not available yet, will work on it).
>
> To not clutter the dialog too much I introduced tabs.
>
> I'm wondering whether it would make sense to merge those two models (RoomsModel and FavoriteRoomsModel) into one (with FavoriteRoomsModel as the base for the new one).
>
>
> This addresses bug 291711.
> http://bugs.kde.org/show_bug.cgi?id=291711
>
>
> Diffs
> -----
>
> dialogs/join-chat-room-dialog.h b5f7c6773f6a73f5653995a559bfab39b085e089
> dialogs/join-chat-room-dialog.cpp 06d0a3ae0d9cbfcb2247dd2593f9a6616613db20
> dialogs/join-chat-room-dialog.ui bec08058e2ffd56255b0f6c7977980226e973390
> rooms-model.h 675ea4cbb02cef5d823b69b6e48b292d3b4b07cf
> rooms-model.cpp 8a5cbb0b8fb0eaed8b30d3b516a574d231d7876e
>
> Diff: http://git.reviewboard.kde.org/r/104173/diff/
>
>
> Testing
> -------
>
> Adding and removing favorites.
> After closing and reopening contact-list the favorites are still there.
>
>
> Screenshots
> -----------
>
> favorites tab
> http://git.reviewboard.kde.org/r/104173/s/448/
> query tab
> http://git.reviewboard.kde.org/r/104173/s/449/
>
>
> Thanks,
>
> Dominik Cermak
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120311/6b5c4a3f/attachment.html>
More information about the KDE-Telepathy
mailing list