Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Battery Docklet UPower Implementation #8

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ammo0110
Copy link

@ammo0110 ammo0110 commented Sep 2, 2019

Hi!

The battery docklet was showing "battery-missing" icon on my laptop. Upon inspection, I found that my machine has "BAT1" as default battery and no "BAT0" was present, but the docklet was only implemented to show "BAT0" information.

To fix this issue, I have written my own battery docklet implementation using UPower DBus interface. I have only implemented the features which are present in the currently existing battery docklet. However, since I am fetching the icon name right from the UPower interface, the icon shows up a bit differently than the currently existing battery docklet.

Please have a look and let me know if there is any improvement to be made.

@bekkra
Copy link

bekkra commented Jul 15, 2021

How could i miss this idea? I solved the same issue but much more brutally. This is the good solution :) How come it is not yet merged?
Also, this is the third commit I see for the same docklet. Where do we discuss this so that we can decide the right move forward?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants