-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110406/
-----------------------------------------------------------

Review request for Plasma.


Description
-------

Depends on Review 110384

This patch makes exposes more battery properties to the Power Management 
dataengine, namely:

- Name: This is the device name of the battery. It will usually be "Unknown 
Battery" but if you eg. attach a mouse (tested with Apple Magic Mouse) or 
another battery device with a name, it will show this. This will eventually 
allow the battery monitor to not just show Battery 1, Battery 2, but actually 
the device name.

- Type: The type of battery, such as Primary (for internal notebook batteries) 
or Mouse. This could also be useful to show eg. a device icon in the battery 
monitor.

- Power Supply: If the battery is actually powering the machine (ie. notebook 
battery) or coming from one of the peripherals (Mouse, Keyboard, ..). This will 
eventually allow the battery monitor to disregard those devices when 
calculating overall battery percentage, or group peripheral devices visually 
separated, or so. 


Diffs
-----

  plasma/generic/dataengines/powermanagement/powermanagementengine.cpp d1b9519 
  plasma/generic/dataengines/powermanagement/powermanagementengine.h 35e9ecf 

Diff: http://git.reviewboard.kde.org/r/110406/diff/


Testing
-------

Tested with Apple Magic Mouse and it properly reported its name and power 
supply state. The Type property is somewhat unreliable as UPower also 
considered the mouse as Primary battery. But the Is Power Supply property was 
correct and that's the prevailing one.


Thanks,

Kai Uwe Broulik

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to