> On July 9, 2015, 8:37 p.m., Martin Klapetek wrote: > > Can you also have a look at https://bugs.kde.org/show_bug.cgi?id=298306 and > > perhaps include the fix for that as well? > > > > Basically the chat applet should collapse when it has nothing to show, just > > like eg. desktop pager or system tray. > > Aleix Pol Gonzalez wrote: > The fix for this bug should end up somewhere else in the code, so it > should be another review. Also I looked into the code and I'm not sure that > the bug is entirely in the plasmoid or in plasma-framework. > > Kai Uwe Broulik wrote: > The pager just sets visible to false on its root item but I didn't manage > to collapse the quick chat that way :/
Ah there's this review of mine which I forgot about which fixes that - https://git.reviewboard.kde.org/r/123445/ - feel free to pick it up, otherwise I'll finish it sometime next week. - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124310/#review82292 ----------------------------------------------------------- On July 9, 2015, 8:03 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124310/ > ----------------------------------------------------------- > > (Updated July 9, 2015, 8:03 p.m.) > > > Review request for Plasma and Telepathy. > > > Repository: ktp-desktop-applets > > > Description > ------- > > This improves the chat widget by: > - Not hardcoding the hader size > - Using a heading for the name so it stands out a bit more (since it's often > not obvious at a glance who you're writing to) > - Giving it a proper scroll bar > - Making the auto scrolling a bit more robust > - Cleaing up the code a bit > > > Diffs > ----- > > chat/org.kde.ktp-chat/contents/ui/ChatWidget.qml 4e802c7 > > Diff: https://git.reviewboard.kde.org/r/124310/diff/ > > > Testing > ------- > > Been using it for a while, works nicely and looks better > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
