nosignal/nosignal-shell
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

internal: fix battery charging logic for tray and lock (#751)

Browse source 
* Update StatusIcons.qml --- Fix battery chargiing logic

idk what the difference is in the return value of UPower.onBattery and UPower.displayDevice.state, but the former always returns false after unplugging my laptop. Changing this to the more specific displayDevice.state === 2 fixes this

* Fetch.qml --- Fix battery charging boolean logic

* Use enum value for UPowerDeviceState instead of index

* Use enum value for UPowerDeviceState instead of index

* Update Fetch.qml --- invert battery conditional logic
This commit is contained in:
Dylan Myers 2025-10-06 04:51:42 -04:00 • committed by GitHub
parent 555539d504
commit 90c051cfa1
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
modules/bar/components/StatusIcons.qml
View file

@ -235,7 +235,7 @@ StyledRect {
}
const perc = UPower.displayDevice.percentage;
const charging = !UPower.onBattery;
const charging = UPower.displayDevice.state === UPowerDeviceState.Charging;
if (perc === 1)
return charging ? "battery_charging_full" : "battery_full";
let level = Math.floor(perc * 7);

2
modules/lock/Fetch.qml
View file

@ -114,7 +114,7 @@ ColumnLayout {
active: UPower.displayDevice.isLaptopBattery
sourceComponent: FetchText {
text: `BATT: ${UPower.onBattery ? "" : "(+) "}${Math.round(UPower.displayDevice.percentage * 100)}%`
text: `BATT: ${UPower.displayDevice.state === UPowerDeviceState.Charging ? "(+) " : ""}${Math.round(UPower.displayDevice.percentage * 100)}%`
}
}
}