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

Commit Hook null at kde.org
Wed Apr 10 22:47:47 UTC 2013


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

(Updated April 10, 2013, 10:47 p.m.)


Status
------

This change has been marked as submitted.


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/plasma-devel/attachments/20130410/b573e9f6/attachment-0001.html>


More information about the Plasma-devel mailing list