> On April 24, 2014, 9:33 a.m., Sebastian Kügler wrote:
> > Screenshots?
> > 
> > My main question is: Is this going to fit into the new systemtray? The 
> > popup is not resizable, so it might push out part of the UI underneath.
> 
> Kai Uwe Broulik wrote:
>     ... which is really unfortunate.
>     
>     However, I will drop the Serial number thing as well as 
> manufacturer/model if it's already clear from the displayed battery name, ie. 
> There's no point in displaying "Logitech Trackball, manufacturer: Logitech".

That's a regression, which will be fixed later. Just pushing in new things that 
will make it appear more broken is not smart.

I'd rather exclude the extra information, in fact I'd rather not make the thing 
expandable at all -- i.e. remove the information. I've this slight suspicion 
that you don't like this suggestion, though. ;)


- Sebastian


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117542/#review56354
-----------------------------------------------------------


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

Reply via email to