----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125669/#review86945 -----------------------------------------------------------
Btw what should be do with the "Show Dashboard" entry in the desktop toolbox? I mean, if you can actually use the toolbox you can probably see your desktop anyway, so we could remove it altogether? - Kai Uwe Broulik On Okt. 17, 2015, 9:51 vorm., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125669/ > ----------------------------------------------------------- > > (Updated Okt. 17, 2015, 9:51 vorm.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > We have a > > connect(KWindowSystem::self(), &KWindowSystem::showingDesktopChanged, > [dashboardAction](bool showing) { > dashboardAction->setText(showing ? i18n("Hide Dashboard") : > i18n("Show Dashboard")); > > in the class constructor. Changing when we're invoked manually is pointless. > > > Diffs > ----- > > shell/shellcorona.cpp b52a7a13b49be184915b1cb350d2e81fe4dffeac > > Diff: https://git.reviewboard.kde.org/r/125669/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
