> On Feb. 8, 2017, 3:47 p.m., Marco Martin wrote: > > any update on this? > > Martin Koller wrote: > Is there any reason this is held back ? > > Marco Martin wrote: > No, could you push this?
Sorry! I have totally missed the updated comments :( Please commit. - Kåre ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129838/#review102476 ----------------------------------------------------------- On Jan. 15, 2017, 2:36 p.m., Martin Koller wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129838/ > ----------------------------------------------------------- > > (Updated Jan. 15, 2017, 2:36 p.m.) > > > Review request for Plasma and Kåre Särs. > > > Bugs: 373776 > http://bugs.kde.org/show_bug.cgi?id=373776 > > > Repository: kdeplasma-addons > > > Description > ------- > > See bug #373776 > The CPU bars do not show a value when using separate bars per CPU, and the > tooltip never > shows a value per CPU, since the data sources per CPU are not subscribed. > > AFAICT this could never have worked. > > > Diffs > ----- > > applets/systemloadviewer/package/contents/ui/CompactBarMonitor.qml 32d98dd > applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml 5a0bc06 > > Diff: https://git.reviewboard.kde.org/r/129838/diff/ > > > Testing > ------- > > yes > > > Thanks, > > Martin Koller > >