Review Request: Handle favorites in JoinChatroomDialog

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104173/#review11391
-----------------------------------------------------------


This review has been submitted with commit caa4e4e53b7074a15936f97730fc875033604f01 by Dominik Cermak to branch master.

- Commit Hook


On March 12, 2012, 7:14 a.m., Dominik Cermak wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104173/
> -----------------------------------------------------------
> 
> (Updated March 12, 2012, 7:14 a.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/20120314/6ff04771/attachment.html>