fix: show configured device name when player has no active stream
Fall back to settings.output_device when the player hasn't reported a device yet (before first playback). Previously showed "No device" even though a device was configured in audio settings. Also change the final fallback from "No device" to "System Default" since no configured device means the system default is being used.
This commit is contained in:
1 file changed
+3
-3
@@ -68,8 +68,8 @@
|
|||||||
const tickerSpeed = 40; // pixels per second
|
const tickerSpeed = 40; // pixels per second
|
||||||
const deviceNameDuration = $derived(deviceNameOverflow > 0 ? `${(deviceNameOverflow + 16) / tickerSpeed}s` : '0s');
|
const deviceNameDuration = $derived(deviceNameOverflow > 0 ? `${(deviceNameOverflow + 16) / tickerSpeed}s` : '0s');
|
||||||
|
|
||||||
// Derived state
|
// Derived state: active stream device, or configured device from settings as fallback
|
||||||
const currentDevice = $derived(outputStatus?.device_name ?? null);
|
const currentDevice = $derived(outputStatus?.device_name ?? settings?.output_device ?? null);
|
||||||
|
|
||||||
// Get PipeWire description for current device
|
// Get PipeWire description for current device
|
||||||
const pipewireDescription = $derived.by(() => {
|
const pipewireDescription = $derived.by(() => {
|
||||||
@@ -94,7 +94,7 @@
|
|||||||
|
|
||||||
// Use PipeWire/ALSA description if available, otherwise fall back to heuristic
|
// Use PipeWire/ALSA description if available, otherwise fall back to heuristic
|
||||||
const prettyDeviceName = $derived(
|
const prettyDeviceName = $derived(
|
||||||
pipewireDescription ?? alsaDescription ?? (currentDevice ? getDevicePrettyName(currentDevice) : 'No device')
|
pipewireDescription ?? alsaDescription ?? (currentDevice ? getDevicePrettyName(currentDevice) : 'System Default')
|
||||||
);
|
);
|
||||||
|
|
||||||
const isExternal = $derived(currentDevice ? isExternalDevice(currentDevice) : false);
|
const isExternal = $derived(currentDevice ? isExternalDevice(currentDevice) : false);
|
||||||
|
|||||||
Reference in new issue
Block a user