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 Status #10

Open
laltm opened this issue Apr 1, 2024 · 11 comments
Open

Battery Status #10

laltm opened this issue Apr 1, 2024 · 11 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@laltm
Copy link

laltm commented Apr 1, 2024

It would be nice to implement the battery charge percentage into the status bar icon.

Maybe like Apple did it on iOS. That way, you could throw out the default icon and save some space in the status bar.
I have no Idea how to do this but I think it would be super neat :)

@mhaeuser
Copy link
Owner

I will not get to this any time soon, but I agree that would be a nice detail. If someone wants to PR this, please make sure to use the event system rather than polling (as done by the daemon) and to not register any events when this feature is currently turned off.

@mhaeuser mhaeuser added enhancement New feature or request help wanted Extra attention is needed labels May 11, 2024
@realrogue
Copy link

@laltm I use Bartender to hide the Battery Toolkit icon as once i have it setup it runs really well in the background, then use iStat to display a more useful battery icon.
iStat has been involved in some controversy lately, so you may want to use one of the alternatives.

@mhaeuser
Copy link
Owner

@rogueda You don’t need to use Bartender, you can simply quit the GUI. All actual logic is in the background daemon.

@laltm
Copy link
Author

laltm commented Jul 23, 2024

some controversy lately
@rogueda
Hi, thanks for the suggestions!
I actually have iStat, but didn't like the way their icon looks. Can you tell me what the controversy was about?

@trevyn
Copy link

trevyn commented Sep 24, 2024

I think Bartender was the one with the controversy.

@franzf
Copy link

franzf commented Oct 20, 2024

@rogueda You don’t need to use Bartender, you can simply quit the GUI. All actual logic is in the background daemon.

Would you be able to add explanation to the GUI somewhere? I was going to leave this running on the menu bar at all times before checking this issue.

@mhaeuser
Copy link
Owner

@franzf This also implies the opposite issue, where people quit the GUI expecting the background process to quit as well. Would a prompt with an explanation that the background activity remains active when quitting resolve your suggestion? You have to try quitting to see it, so it's not perfect.

@alexisgaziello
Copy link

@franzf This also implies the opposite issue, where people quit the GUI expecting the background process to quit as well. Would a prompt with an explanation that the background activity remains active when quitting resolve your suggestion? You have to try quitting to see it, so it's not perfect.

I think that would be an cool addition. I wasn't aware of it. And maybe even a toggle to keep the daemon on/off when quitting the app.

@mhaeuser
Copy link
Owner

Such a toggle would be dirty, because the GUI is a user process and the daemon is a root process (shared across all users). What I could and might do is adding a “Pause Background Activity” command to the menu, but so far nobody really asked for it or provided a use-case.

@alexisgaziello
Copy link

Can you turn off the daemon without uninstalling?

@mhaeuser
Copy link
Owner

From System Settings > Login Items, yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

6 participants