> On March 16, 2014, 4:11 p.m., Sebastian Kügler wrote: > > plasma/desktop/applets/kickoff/package/contents/ui/Kickoff.qml, line 185 > > <https://git.reviewboard.kde.org/r/116836/diff/1/?file=254503#file254503line185> > > > > This is going to raise a warning b/c you're assigning undefined to an > > int, no? Returning 0 here is fine and avoids that warning. > > > > Same for the height.
Yeah made me suspicious that the original code (in the switch default) did that which I just adopted - "I guess they know what they're doing" mode - Kai Uwe ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116836/#review53066 ----------------------------------------------------------- On March 16, 2014, 3:54 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 16, 2014, 3:54 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
