From ab319a96f223163696ce32e1425665df0afde7f9 Mon Sep 17 00:00:00 2001
From: ComixHe <heyuming@deepin.org>
Date: Mon, 2 Feb 2026 18:20:47 +0800
Subject: [PATCH] fix(display): prevent unintended brightness reset to 10%

- Replace onValueChanged with onMoved for brightness slider to ensure
  backend values are only updated by explicit user interaction.
- Fix a race condition where slider auto-correction (to 'from' value 0.1)
  would overwrite C++ data during object destruction.

Pms: BUG-308187 BUG-307119
Log: prevent unintended brightness reset to 10%
Signed-off-by: ComixHe <heyuming@deepin.org>
---
 src/plugin-display/qml/DisplayMain.qml | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/plugin-display/qml/DisplayMain.qml b/src/plugin-display/qml/DisplayMain.qml
index 31f219b0a0..e648a79544 100644
--- a/src/plugin-display/qml/DisplayMain.qml
+++ b/src/plugin-display/qml/DisplayMain.qml
@@ -540,10 +540,8 @@ DccObject {
             to: 1
             stepSize: 0.01
             value: screenItem.brightness
-            onValueChanged: {
-                if (screenItem.brightness !== value) {
-                    screenItem.brightness = value
-                }
+            onMoved: {
+                screenItem.brightness = value
             }
         }
         D.DciIcon {