----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116836/#review53306 -----------------------------------------------------------
Ship it! Good to go in. Thanks for taking care of it. You can mark the "issues" as Fixed as you go, I usually do that, using them as helper to track the issues I need to fix in the patch. - Sebastian Kügler On March 17, 2014, 7:58 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116836/ > ----------------------------------------------------------- > > (Updated March 17, 2014, 7:58 p.m.) > > > Review request for Plasma and Sebastian Kügler. > > > Repository: kde-workspace > > > Description > ------- > > This patch expands the search results list to cover the entire height (or > horizontal space in case of left/right panel) of the plasmoid since the > tabbar is hidden and it looks weird to reserve space for something that isn't > there at the moment. > > It also animates the re-appearance of the tabbar. Disappearance/Opacity of > the tabbar is not animated due to performance reasons (got heavy lagging). > > > Diffs > ----- > > plasma/desktop/applets/kickoff/package/contents/ui/Kickoff.qml bfb1d16 > > Diff: https://git.reviewboard.kde.org/r/116836/diff/ > > > Testing > ------- > > Tested in bottom, top, left and right panel and repeatedly typed in > characters and removed them again :) > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
