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

[Commit Hook]

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


This review has been submitted with commit ad880faaa54d05c4ccaf64e90d7b793682871084 by Aleix Pol to branch master.

- Commit Hook


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/0e440542/attachment.html>