Review Request: Improve Chat Plasmoid size behavior

[Lasath Fernando]

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

Ship it!


Neat patch. 
I do have a question though - what would this do if there are more active chats than can fit in a panel?


plasmoid/org.kde.ktp-chatplasmoid/contents/ui/main.qml
<http://git.reviewboard.kde.org/r/104633/#comment9840>

    I think according to the kde style guide, you're supposed to pad operators.
    
    i.e. put space before and after the '>'



plasmoid/org.kde.ktp-chatplasmoid/contents/ui/main.qml
<http://git.reviewboard.kde.org/r/104633/#comment9839>

    very cool ;-)


- Lasath Fernando


On April 17, 2012, 12:01 a.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104633/
> -----------------------------------------------------------
> 
> (Updated April 17, 2012, 12:01 a.m.)
> 
> 
> Review request for Telepathy and David Edmundson.
> 
> 
> Description
> -------
> 
> This patch is quite cool.
>  - Report the list size as the minimum size
>  - If its shape is vertical (width<height) set a vertical orientation. This lets you put the plasmoid on a vertical panel
> 
> 
> Diffs
> -----
> 
>   plasmoid/org.kde.ktp-chatplasmoid/contents/ui/main.qml f548c33 
>   plasmoid/org.kde.ktp-chatplasmoid/metadata.desktop f949f65 
> 
> Diff: http://git.reviewboard.kde.org/r/104633/diff/
> 
> 
> Testing
> -------
> 
> Talked to myself a bit, again.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120417/1e342f81/attachment.html>