Review Request: Maintain a list of recently used rooms in JoinChatroomDialog

[Dominik Cermak]

> On March 14, 2012, 11:18 a.m., Vishesh Handa wrote:
> > dialogs/join-chat-room-dialog.cpp, line 52
> > <http://git.reviewboard.kde.org/r/104269/diff/1/?file=53074#file53074line52>
> >
> >      I know this was like this before, but why have the closing parenthesis on the next line?

Don't know ;-)
It was this way before I started working on this file so I let it.


- Dominik


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


On March 14, 2012, 11 a.m., Dominik Cermak wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104269/
> -----------------------------------------------------------
> 
> (Updated March 14, 2012, 11 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Upon joining a room (more exact upon clicking "Ok") the room is saved in the recently used list for this account.
> ATM it remembers up to 8 rooms and single rooms can be removed as well as the whole list can be cleared.
> 
> The complete list is showed in a new tab and the rooms are used for completion in the lineEdit.
> 
> 
> This addresses bug 291711.
>     http://bugs.kde.org/show_bug.cgi?id=291711
> 
> 
> Diffs
> -----
> 
>   dialogs/join-chat-room-dialog.h 686fe312dc243e49feafa793106f782abb9f9f9b 
>   dialogs/join-chat-room-dialog.cpp 026c5e4f3d1e068739c40a4b014daf84e8a77f8b 
>   dialogs/join-chat-room-dialog.ui 839d644399b5c9031a4c4840c6e8573309c4b859 
> 
> Diff: http://git.reviewboard.kde.org/r/104269/diff/
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> recent tab
>   http://git.reviewboard.kde.org/r/104269/s/462/
> recent completion
>   http://git.reviewboard.kde.org/r/104269/s/463/
> 
> 
> Thanks,
> 
> Dominik Cermak
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120314/55a96ae1/attachment-0001.html>