Fix Bluetooth panel device list handling

This commit is contained in:
Ryan Hughes
2026-05-28 15:04:41 -04:00
parent 801c2b00c8
commit 007393387a
3 changed files with 42 additions and 9 deletions
+15 -2
View File
@@ -3,6 +3,18 @@ function deviceLabel(device) {
return String(device.deviceName || device.name || "").trim()
}
function toArray(values) {
if (!values) return []
if (Array.isArray(values)) return values.slice()
var length = Number(values.length || 0)
if (!isFinite(length) || length <= 0) return []
var list = []
for (var i = 0; i < length; i++) list.push(values[i])
return list
}
function isUuidLike(value) {
var text = String(value || "").trim()
if (text === "") return false
@@ -23,13 +35,13 @@ function hasHumanName(device) {
}
function sortedByLabel(devices) {
var list = Array.isArray(devices) ? devices.slice() : []
var list = toArray(devices)
list.sort(function(a, b) { return deviceLabel(a).localeCompare(deviceLabel(b)) })
return list
}
function deviceLists(devices) {
var values = Array.isArray(devices) ? devices : []
var values = toArray(devices)
var connected = []
var known = []
var discovered = []
@@ -86,6 +98,7 @@ function sectionDevices(lists, section) {
if (typeof module !== "undefined") {
module.exports = {
deviceLabel: deviceLabel,
toArray: toArray,
isUuidLike: isUuidLike,
isAddressLike: isAddressLike,
hasHumanName: hasHumanName,
+6 -7
View File
@@ -219,9 +219,9 @@ Panel {
function moveCursorH(delta) {
if (!cursorActive) { cursorActive = true; return }
if (focusSection !== "known") return
if (focusSection !== "known" && focusSection !== "connected") return
var dev = deviceAt(focusSection, selectedIndex)
if (!dev || dev.connected) return
if (!dev || !dev.address) return
if (delta > 0) actionFocused = true
else if (delta < 0) actionFocused = false
}
@@ -246,11 +246,10 @@ Panel {
}
}
// 'x' forgets remembered devices. Connected rows toggle connection via
// Enter/click, so the destructive forget action is intentionally unavailable
// while connected.
// 'x' forgets remembered devices. For connected devices this first
// disconnects, then removes the BlueZ pairing record via omarchy-bluetooth-device.
function deleteSelected() {
if (focusSection !== "known") return
if (focusSection !== "known" && focusSection !== "connected") return
var dev = deviceAt(focusSection, selectedIndex)
if (!dev) return
forgetDevice(dev)
@@ -626,7 +625,7 @@ Panel {
}
readonly property bool rowSelected: root.cursorActive && root.focusSection === sectionName && root.selectedIndex === rowIndex
readonly property bool forgetAvailable: sectionName === "known" && !isConnected && !isDiscovered
readonly property bool forgetAvailable: (sectionName === "known" || sectionName === "connected") && !isDiscovered
readonly property bool showForgetButton: forgetAvailable && (rowMouse.containsMouse || rowSelected)
hasCursor: rowSelected && !root.actionFocused