Fix an issue with non-zero exit code states in the setup, add Rokid Max 2 to the supported models list, fix the connection status in the UI, fix an issue with the negative values on the labeled sliders

bin/breezy_kwin_setup

@@ -118,17 +118,16 @@ then
     pkgkwin_stderr=$(LOCAL_BUILD_SYSTEM=1 bin/package_kwin --download-driver 2>&1)
     pkgkwin_rc=$?
     set -e
-    if echo "$pkgkwin_stderr" | grep -qi "could not find"; then
-      if [ -z "${PRINT_FULL_STDERR+x}" ]; then
-        print_missing_dependencies "$0"
-        exit 1
-      else
-        echo "$pkgkwin_stderr"
-      fi
-    fi
     if [ "$pkgkwin_rc" -ne 0 ]; then
-      echo "$pkgkwin_stderr"
-      echo "Error: build failed with exit code $pkgkwin_rc"
+      if echo "$pkgkwin_stderr" | grep -qi "could not find"; then
+        if [ -z "${PRINT_FULL_STDERR+x}" ]; then
+          print_missing_dependencies "$0"
+        else
+          echo "$pkgkwin_stderr"
+        fi
+      fi
+      echo ""
+      echo "Error: Breezy Desktop build failed with exit code $pkgkwin_rc"
       exit $pkgkwin_rc
     fi
     FILE_NAME="breezyKWin-$ARCH.tar.gz"

gnome/src/monitormanager.js

@@ -34,6 +34,7 @@ export const SUPPORTED_MONITOR_PRODUCTS = [
     'Air 2 Ultra',
     'SmartGlasses', // TCL/RayNeo
     'Rokid Max',
+    'Rokid Max 2',
     'Rokid Air',
     NESTED_MONITOR_PRODUCT
 ];

kwin/src/kcm/breezydesktopeffectkcm.cpp

@@ -191,9 +191,9 @@ void BreezyDesktopEffectConfig::pollDriverState()
 
     const bool wasDeviceConnected = m_deviceConnected;
     m_deviceConnected = !m_connectedDeviceBrand.isEmpty() && !m_connectedDeviceModel.isEmpty();
-    if (m_deviceConnected != wasDeviceConnected) {
-        ui.labelDeviceConnectionStatus->setText(m_deviceConnected ? 
-            QStringLiteral("%1 %2 connected").arg(m_connectedDeviceBrand, m_connectedDeviceModel) : 
+    if (ui.labelDeviceConnectionStatus->text().isEmpty() || m_deviceConnected != wasDeviceConnected) {
+        ui.labelDeviceConnectionStatus->setText(m_deviceConnected ?
+            QStringLiteral("%1 %2 connected").arg(m_connectedDeviceBrand, m_connectedDeviceModel) :
             QStringLiteral("No device connected"));
     }
 

kwin/src/kcm/breezydesktopeffectkcm.ui

@@ -2,14 +2,11 @@
 <ui version="4.0">
  <class>BreezyDesktopEffectConfig</class>
  <widget class="QWidget" name="BreezyDesktopEffectConfig">
-  <property name="minimumWidth">
-   <number>800</number>
-  </property>
-    <layout class="QVBoxLayout" name="verticalLayout">
-     <item>
-        <widget class="QLabel" name="labelDeviceConnectionStatus">
-         <property name="text">
-            <string>No device connected</string>
+  <layout class="QVBoxLayout" name="verticalLayout">
+   <item>
+      <widget class="QLabel" name="labelDeviceConnectionStatus">
+       <property name="text">
+          <string></string>
          </property>
          <property name="alignment">
             <set>Qt::AlignHCenter|Qt::AlignVCenter</set>

kwin/src/kcm/labeledslider.h

@@ -134,11 +134,13 @@ private:
             return QString::number(raw);
         }
         int divisor = 1;
-        for (int i = 0; i < m_decimalShift; ++i) divisor *= 10; // small loop, m_decimalShift capped
+        for (int i = 0; i < m_decimalShift; ++i) divisor *= 10;
         int whole = raw / divisor;
         int frac = std::abs(raw % divisor);
         QString fracStr = QString::number(frac).rightJustified(m_decimalShift, QLatin1Char('0'));
-        return QString::number(whole) + QLatin1Char('.') + fracStr;
+        QString result = QString::number(std::abs(whole)) + QLatin1Char('.') + fracStr;
+        if (raw < 0) result.prepend(QLatin1Char('-'));
+        return result;
     }
 
     bool m_showValueBubble = true;

kwin/src/qml/main.qml

@@ -17,6 +17,7 @@ Item {
         "Air 2 Ultra",
         "SmartGlasses", // TCL/RayNeo
         "Rokid Max",
+        "Rokid Max 2",
         "Rokid Air"
     ]
     required property QtObject effect