Those information mostly clutter the plasmoid UI and are not very useful in the energy plasmoid
The battery plasmoid represents the state of the battery, for battery information we have the Energy information KCM in kinfocenter that will display this after D23152 .
Details
Before:
After:
Diff Detail
- Repository
- R120 Plasma Workspace
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 15250 Build 15268: arc lint + arc unit
You cannot remove those from the dataengine, which is "Public API", there might be other users of that.
Instead, remove them from the battery monitor UI. And while at it maybe show the details always even if there are multiple batteries.
Capacity is also used in battery monitor to show a hint like "Your battery is broken"
Also, this is not the network plasmoid but battery.
Furthermore, I would appreciate if you could wait for maintainer approval before pushing changes that are clearly wrong, not everyone spends Sunday afternoon at a computer.
In which GUI do I find "Vendor" and "Model" now? There's the "Energy Information" in KInfocenter, which
has a section "manufacturer" (in screenshot "Hersteller"), but only lists the serial number. Maby it could go there?
Is this information really that cluttering to mandate removal? Sure, vendor and model may be too "advanced" but remaining capacity is an useful hint.
I always liked this sort of stuff when coming from Windows.
I would argue that capacity counts as "state".
My intention is by no means to hide this data but move it somewhere more discreet proportionate to its usefulness.
We might want to keep it in the plasmoid but only when capacity < 90 for instance.
I always liked this sort of stuff when coming from Windows.
Differentiating from windows is not a reason to add or keep features.
Missed that.
Also, this is not the network plasmoid but battery.
Furthermore, I would appreciate if you could wait for maintainer approval before pushing changes that are clearly wrong, not everyone spends Sunday afternoon at a computer.
My bad, I overreacted, three approvals are kind of rare.
In the meantime I have reverted this patch.