> On Juli 9, 2015, 6:37 nachm., 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.
The pager just sets visible to false on its root item but I didn't manage to collapse the quick chat that way :/ - Kai Uwe ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124310/#review82292 ----------------------------------------------------------- On Juli 9, 2015, 6:03 nachm., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124310/ > ----------------------------------------------------------- > > (Updated Juli 9, 2015, 6:03 nachm.) > > > 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
