Review Request 109947: KTp Chat plasmoid: Improve integration with hiding panels

David Edmundson david at davidedmundson.co.uk
Wed Apr 10 14:49:23 UTC 2013


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

Ship it!


Ship It!

- David Edmundson


On April 10, 2013, 2:39 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109947/
> -----------------------------------------------------------
> 
> (Updated April 10, 2013, 2:39 p.m.)
> 
> 
> Review request for Plasma, Telepathy and David Edmundson.
> 
> 
> Description
> -------
> 
> This patch changes 2 things:
> - changes the code from a js function to setting a qml property. This is better because we don't have to connect to all the changing signals to re-evaluate (there were some weird problems in this regard).
> - Also it makes it so when you have a dialog opened, the status is set to AcceptingInputStatus, this way the panel doesn't hide while you're writing. This is useful because this way you can see if somebody talks to you while you're chatting already.
> 
> 
> Diffs
> -----
> 
>   chat/org.kde.ktp-chat/contents/ui/main.qml 9fb78c9 
> 
> Diff: http://git.reviewboard.kde.org/r/109947/diff/
> 
> 
> Testing
> -------
> 
> Self-conversation, as usual.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

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


More information about the KDE-Telepathy mailing list