feat(control-center): show Bluetooth battery in quick tile - #3160
Conversation
|
I can make it toggleable from the settings if you want. But it utilizes free space, so I see 0 problems, only value :) |
|
/claude review |
| if (primaryDevice.batteryAvailable) | ||
| return `${name} • ${Math.round(primaryDevice.battery * 100)}%`; |
There was a problem hiding this comment.
Missing the battery > 0 guard the rest of the codebase uses (BluetoothService.qml:75, BluetoothDetail.qml:365). BlueZ reports batteryAvailable true with battery == 0 for devices that don't actually publish a level, so this shows Device • 0% and returns early, skipping the UPower fallback below that may have the real value.
| if (primaryDevice.batteryAvailable) | |
| return `${name} • ${Math.round(primaryDevice.battery * 100)}%`; | |
| if (primaryDevice.batteryAvailable && primaryDevice.battery > 0) | |
| return `${name} • ${Math.round(primaryDevice.battery * 100)}%`; |
Claude reviewLooks good — one minor consistency issue, no blockers.
Checked: binding reactivity of |
85de4e8 to
c1aeff4
Compare
|
/claude review |
Display the selected device as `name • N%` when battery information is available, while preserving the existing name-only fallback.
c1aeff4 to
3616fea
Compare
|
@bbedward I've added a comment with links to the |
|
/claude review |
Claude reviewNo issues found. The Checked: binding reactivity of |
Description
Shows the connected Bluetooth device battery level in the Control Center tile when available.
Type of change
Related issues
None.
Screenshots / video
Checklist
make lint-qmlwith no new warnings