Review Request 109268: Combining settings in a singleton class so that there is no confusion about the current settings.

Stefan Eggers coloncolonone at gmail.com
Mon Mar 4 11:10:32 UTC 2013



> On March 3, 2013, 9 p.m., David Edmundson wrote:
> > Please join our mailing list, otherwise don't get notified about new review requests.
> > 
> > https://mail.kde.org/mailman/listinfo/kde-telepathy
> > 
> > Looks pretty good :) There's some minor comments.
> > Do you have KDE developer access, or do you need one of us to commit for you?
> >
> 
> David Edmundson wrote:
>     Edit - it just came through.. ignore my mailing list comment. My client must just be slow.

No, I don't have developer access. So someone else would have to do the actual commit.


- Stefan


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


On March 4, 2013, 11:06 a.m., Stefan Eggers wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109268/
> -----------------------------------------------------------
> 
> (Updated March 4, 2013, 11:06 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Combining settings in a singleton class so that there is no confusion about the current settings.
> 
> Up to now the settings for where new chats open were loaded by
> ktp-text-ui on startup and not re-read anymore. Changing the settings
> in the config dialog had no immediate effect; only after the next
> start of ktp-text-ui it finally changed.
> 
> Now whenever the settings get asked for the singleton provides the
> current value. Thus every change to the settings via the config dialog
> will take effect immediately.
> 
> Also gives a way to easily add new settings in one place. Will be
> useful for implementing the additional settings required by bug 282201
> (make it configurable if "user is typing" gets show") which needs two
> more bools.
> 
> 
> This addresses bug 316004.
>     http://bugs.kde.org/show_bug.cgi?id=316004
> 
> 
> Diffs
> -----
> 
>   app/telepathy-chat-ui.h 70800f42a4dd8f5627495661a2f9c71535f20763 
>   app/telepathy-chat-ui.cpp 920210891e58166d1c14f5431a913a8e5f912de2 
>   config/behavior-config.h 562187ee308182f70b66b09ecf489a13fbfb6fdd 
>   config/behavior-config.cpp 214ee3e5641185d200c9aca2892aee78e9915e5b 
>   lib/CMakeLists.txt 4e03671dcdaee338aca57dae12512b75d7f0be15 
>   lib/KTp/TextChatConfig PRE-CREATION 
>   lib/text-chat-config.h PRE-CREATION 
>   lib/text-chat-config.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/109268/diff/
> 
> 
> Testing
> -------
> 
> 1. Starting with "Open new conversations:" set to "As tabs in the same window".
> 2. Open a chat with user A.
> 3. Open a chat with user B.
> 4. Observe that it opens in a new tab.
> 5. Change setting to "As new windows".
> 6. Close chat with user B.
> 7. Open a chat with user B.
> 8. Observe that it opens in a new window.
> 
> 
> Thanks,
> 
> Stefan Eggers
> 
>

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


More information about the KDE-Telepathy mailing list