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


Like suggested David on IRC, I tried to use the currentIndex to define which 
one is shown. Patch: http://paste.kde.org/456494/

The problem with this is that we need make sure that just 1 is checked while 
the checked property is changed by onClicked itself, so we can't really attach 
this value to another property because the value ends up overriden. We might be 
able to fix this by making a hack (adding a checkedButton and exclusive 
property to make it think we're a button row) but in the end I think that the 
patch proposed here is straightforward enough.

- Aleix Pol Gonzalez


On April 13, 2012, 1:14 a.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104575/
> -----------------------------------------------------------
> 
> (Updated April 13, 2012, 1:14 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Only display 1 chat dialog at the same time. If a different contact is 
> clicked, close the previously shown.
> 
> 
> Diffs
> -----
> 
>   plasmoid/org.kde.ktp-chatplasmoid/contents/ui/main.qml 3a9b71f 
> 
> Diff: http://git.reviewboard.kde.org/r/104575/diff/
> 
> 
> Testing
> -------
> 
> Roughly tried to reproduce the use case myself.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

_______________________________________________
KDE-Telepathy mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-telepathy

Reply via email to