----------------------------------------------------------- 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 > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
