Review Request 109948: KTp Chat plasmoid: Improve reaction to closing dialog

[David Edmundson]

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



chat/org.kde.ktp-chat/contents/ui/ChatWidget.qml
<http://git.reviewboard.kde.org/r/109948/#comment22949>

    I don't understand the patch. 
    
    As I understand it requestClose() invalidates the channel. When the channel is invalided the ConversationsModel removes it.
    
    When the model removes it the delegate should be deleted. Right?
    


In addition closeRequested and requestClose() doing completely different things is very confusing.

- David Edmundson


On April 10, 2013, 3:50 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109948/
> -----------------------------------------------------------
> 
> (Updated April 10, 2013, 3:50 p.m.)
> 
> 
> Review request for Telepathy and David Edmundson.
> 
> 
> Description
> -------
> 
> Delegate deletion happens after deletion of the data, so we don't have the index available anymore. This makes it so when we close the dialog the UI doesn't react appropriately.
> 
> This patch copies our code to react to the delegate deletion (hidding the dialog) before the actual deletion happens.
> 
> 
> Diffs
> -----
> 
>   chat/org.kde.ktp-chat/contents/ui/ChatWidget.qml 10e7d43 
> 
> Diff: http://git.reviewboard.kde.org/r/109948/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

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