----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120961/#review69757 -----------------------------------------------------------
Ship it! Inviala! - Marco Martin On Nov. 3, 2014, 8:47 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120961/ > ----------------------------------------------------------- > > (Updated Nov. 3, 2014, 8:47 p.m.) > > > Review request for Plasma. > > > Repository: plasma-desktop > > > Description > ------- > > Panelcontroller already takes quite some time to load, so we should not load > the "More Settings" menu if it is not opened. > > I usually only open the panelcontroller to quickly remove an applet I > accidentally added to my panel, I hardly ever use the settings menu. > > That import org.kde.plasma.configuration 2.0 looks unused to me. > > > Diffs > ----- > > > desktoppackage/contents/configuration/panelconfiguration/MoreSettingsMenu.qml > PRE-CREATION > desktoppackage/contents/configuration/panelconfiguration/Ruler.qml c4c2ed3 > desktoppackage/contents/configuration/panelconfiguration/ToolBar.qml > 828147c > > Diff: https://git.reviewboard.kde.org/r/120961/diff/ > > > Testing > ------- > > There's still a noticeable delay but it's less than before though I hoped > that patch helped more significantly. Other than that works as expected. > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
