----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117542/#review56298 -----------------------------------------------------------
applets/batterymonitor/contents/code/logic.js <https://git.reviewboard.kde.org/r/117542/#comment39358> Can you explain this change? applets/batterymonitor/contents/ui/BatteryItem.qml <https://git.reviewboard.kde.org/r/117542/#comment39359> it would be good to see if we can use something better than != 0 0 is a perfectly valid temperature. For two batteries an energy rate could be 0. using !== might work out well. Does it still work with Alex's changes? - David Edmundson On April 13, 2014, 1:45 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117542/ > ----------------------------------------------------------- > > (Updated April 13, 2014, 1:45 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > This adds the following improvements provided by Review 117339 and Review > 117540: > - Show temperature, voltage, energy rate and serial number. Since those are > only shown when expanding the battery it's imho not that bad to show some > geeky stuff which can give you some more indication of battery health and > overall power consumption. It adds new strings, however. It also changes the > substr hackery to "enabled: false" for greying out the labels. > > What's the preferred way of formatting numbers (decimal points, units)? The > formats thing only seems to be for file sizes and durations. > > > Diffs > ----- > > applets/batterymonitor/contents/code/logic.js 5b4cb52 > applets/batterymonitor/contents/ui/BatteryItem.qml 431aa9a > applets/batterymonitor/contents/ui/CompactRepresentation.qml 39cd331 > applets/batterymonitor/contents/ui/DetailsLabel.qml d973d22 > applets/batterymonitor/contents/ui/PopupDialog.qml 0444199 > applets/batterymonitor/contents/ui/batterymonitor.qml 4711c5e > > Diff: https://git.reviewboard.kde.org/r/117542/diff/ > > > Testing > ------- > > Compiles. Information is properly displayed when expanding a battery. > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel