Joren/qbz
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(slint): queue sleep-timer flyout closed on any click

Browse Source 
The sleep-timer PopupWindow used the default close-policy (close-on-click),
so any click inside it — including on the preset picker — closed it before
the value could change. Set close-policy: close-on-click-outside. Also
replaced the QbzSelect (itself a PopupWindow, which Slint cannot open nested
inside another PopupWindow) with inline preset rows, so the presets are
actually pickable.

cc
This commit is contained in:
vicrodh committed 2026-06-29 16:58:16 -06:00
1 parent 6f6f2d5aec
commit e0e6ca81a9
1 file changed
+32 -7
Download Patch File Download Diff File Expand all files Collapse all files
crates/qbz-ui/ui/shell/QueueSidebar.slint
+32 -7
View File
@@ -14,7 +14,6 @@ import { Spacing } from "../foundation/spacing.slint";
import { Typography } from "../foundation/typography.slint";
import { Radius } from "../foundation/radius.slint";
import { QbzIcon } from "../primitives/QbzIcon.slint";
import { QbzSelect } from "../primitives/QbzSelect.slint";
import { ContextMenu } from "../primitives/ContextMenu.slint";
import { ContextMenuItem } from "../primitives/ContextMenuItem.slint";
import { ShellState, QueueState, QueueItem, NowPlayingState, UiFocusState, SleepTimerState, SleepTimerActions } from "../state.slint";
@@ -807,6 +806,10 @@ export component QueueSidebar inherits Rectangle {
x: -188px;
y: -214px;
width: 220px;
// Default close-policy is close-on-click, which closed the
// flyout on ANY click inside it (you couldn't pick a preset
// without it closing first). Only an outside click closes.
close-policy: close-on-click-outside;
Rectangle {
width: 220px;
border-radius: Radius.md;
@@ -864,12 +867,34 @@ export component QueueSidebar inherits Rectangle {
color: Theme.text-muted;
font-size: 11px;
}
QbzSelect {
options: [@tr("30 min"), @tr("1 hr"), @tr("2 hr"), @tr("3 hr"), @tr("5 hr"), @tr("Custom…")];
current-index: SleepTimerState.selected-index;
sm: true;
selected(idx) => {
SleepTimerState.selected-index = idx;
// Inline preset rows. A QbzSelect here would nest a
// PopupWindow inside this PopupWindow (Slint can't
// open that), so the list is rendered inline.
VerticalLayout {
spacing: 2px;
for opt[i] in [@tr("30 min"), @tr("1 hr"), @tr("2 hr"), @tr("3 hr"), @tr("5 hr"), @tr("Custom…")]: Rectangle {
height: 30px;
border-radius: Radius.sm;
background: SleepTimerState.selected-index == i
? Theme.accent
: (preset-ta.has-hover ? Theme.surface-hover : transparent);
Text {
x: 10px;
width: parent.width - 20px;
height: 100%;
text: opt;
color: SleepTimerState.selected-index == i
? Theme.accent-text
: Theme.text-primary;
font-size: 12px;
vertical-alignment: center;
}
preset-ta := TouchArea {
mouse-cursor: pointer;
clicked => {
SleepTimerState.selected-index = i;
}
}
}
}
Rectangle {