search

Powerlevel9k / powerlevel9k

Powerlevel9k was a tool for building a beautiful and highly functional CLI, customized for you. P9k had a substantial impact on CLI UX, and its legacy is now continued by P10k.
https://github.com/romkatv/powerlevel10k
MIT License
13.46k stars 949 forks source link

multiple batteries #1379

Open tibix opened 4 years ago

tibix commented 4 years ago

@dritter I found a "limitation" related to laptops that have more than 1 battery. It looks like the code looks only at BAT0 or BAT1 to display the capacity. However, I'm using a couple of ThinkPads now that have 2 batteries, and is confusing when one battery runs low and the system switched to the second battery, the prompt still show the details for the first battery. On my system I managed to overcome the limitation with the following addition:

if [[ -a $sysp/BAT0 ]] && [[ -a $sysp/BAT0 ]]; then
    local bat0=$(cat $sysp/BAT0/capacity)
    local bat1=$(cat $sysp/BAT1/capacity)

    local capacity=$((($bat0 + $bat1)/2))
else
    local capacity=$(cat $bat/capacity)
fi

If you feel like this is something worth added in the next release ... feel free to use the code above, or if I can do something, please let me know.

Syphdias commented 4 years ago

This is actually implemented on the next branch already. I am not sure when it will be merged into master