From 70a01d7087ba933087cc5c23ecd8a8135d4fe703 Mon Sep 17 00:00:00 2001 From: rickpeters Date: Mon, 28 Sep 2026 19:03:56 +0200 Subject: [PATCH] =?UTF-8?q?fix(ui):=20The=20progress=20screen's=20?= =?UTF-8?q?=E2=9C=93=20and=20!=20stand=20upright:=20only=20a=20dot=20on=20?= =?UTF-8?q?the=20circle=20spins=20while=20a=20step=20runs=20(they=20kept?= =?UTF-8?q?=20the=20spinner's=20last=20angle)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- CHANGELOG.md | 3 ++- ui/qml/ApplyingScreen.qml | 6 +++++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 8b187f4..bab3f54 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,7 +7,8 @@ one per merged pull request. ## 2.9.0 - 2026-09-28 -- **chore: Version 2.9.0** +- `d30d505` **chore: Version 2.9.0** +- **fix(ui): The progress screen's ✓ and ! stand upright: only a dot on the circle spins while a step runs (they kept the spinner's last angle)** ## 2.8.0 - 2026-09-28 diff --git a/ui/qml/ApplyingScreen.qml b/ui/qml/ApplyingScreen.qml index 56c69b6..64a5d7a 100644 --- a/ui/qml/ApplyingScreen.qml +++ b/ui/qml/ApplyingScreen.qml @@ -35,7 +35,11 @@ Item { border.width: step.st === "wait" || step.st === "run" ? 2 : 0; border.color: step.st === "run" ? Theme.accent : "#343f50" Text { anchors.centerIn: parent; text: step.st === "ok" ? "✓" : (step.st === "fail" ? "!" : ""); color: step.st === "ok" ? Theme.good : Theme.bad font.pixelSize: 13; font.weight: Font.Bold } - RotationAnimator on rotation { running: step.st === "run"; from: 0; to: 360; duration: 1000; loops: Animation.Infinite } } + // Only this dot turns while the step runs: ✓ and ! never rotate + // (an animator leaves its item at whatever angle it stopped). + Item { anchors.fill: parent; visible: step.st === "run" + Rectangle { width: 6; height: 6; radius: 3; color: Theme.accent; x: parent.width / 2 - 3; y: -3 } + RotationAnimator on rotation { running: step.st === "run"; from: 0; to: 360; duration: 1000; loops: Animation.Infinite } } } Row { width: 380; anchors.verticalCenter: parent.verticalCenter; spacing: 10 // Boot into's label is its prefix already: "Boot into desktop", not "Boot into Boot into". Text { text: step.modelData.id === "boot" ? (Texts.actions[step.modelData.action] || [])[3] + Texts.actions[step.modelData.action][0].toLowerCase()