Review Request 108571: Allow a separate config for group chats

David Edmundson david at davidedmundson.co.uk
Fri Mar 15 11:23:22 UTC 2013



> On March 15, 2013, 9:18 a.m., Martin Klapetek wrote:
> > config/appearance-config-tab.cpp, line 177
> > <http://git.reviewboard.kde.org/r/108571/diff/4/?file=119832#file119832line177>
> >
> >     Should this be translatable?

Apparently all strings should, I didn't understand why I just got told off before


- David


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


On March 15, 2013, 1:46 a.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108571/
> -----------------------------------------------------------
> 
> (Updated March 15, 2013, 1:46 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Allow a separate config for group chats
> 
> Work by our code-in student Huba Nagy. I've not reviewed it myself.
> 
> 
> Diffs
> -----
> 
>   app/chat-window.cpp 9a014e35c67d20b8fa3fc98049f7e2fb3f4a5317 
>   config/CMakeLists.txt 24e9e80ffd9c23515a457a5d2f0ba8271e207e17 
>   config/appearance-config-tab.h PRE-CREATION 
>   config/appearance-config-tab.cpp PRE-CREATION 
>   config/appearance-config.h 48bd67aaad5e31c7e5409447b2d4f807f9542748 
>   config/appearance-config.cpp 247b47defb151f46d5a2f672002a1096e7567e91 
>   config/appearance-config.ui a6d55a2d9d0e9ae4b1b05fb389893c07c2fbf8d2 
>   lib/adium-theme-view.h 5775432619e3b9c3759221fb1a91780cd4583079 
>   lib/adium-theme-view.cpp f58beb630966ed046d2a90381b526cd0945dc8fe 
>   lib/chat-widget.cpp 6cfbce60ce4d924f53585eb329074d64e504846c 
> 
> Diff: http://git.reviewboard.kde.org/r/108571/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

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


More information about the KDE-Telepathy mailing list