Revert 8c81514

Modules/Bar/Bar.qml

@@ -92,7 +92,7 @@ Variants {
 
             // Top section (left widgets)
             Column {
-              spacing: Style.marginXXS * root.scaling
+              spacing: Style.marginS * root.scaling
               anchors.horizontalCenter: parent.horizontalCenter
               anchors.top: parent.top
               anchors.topMargin: Style.marginM * root.scaling
@@ -118,7 +118,7 @@ Variants {
 
             // Center section (center widgets)
             Column {
-              spacing: Style.marginXXS * root.scaling
+              spacing: Style.marginS * root.scaling
               anchors.horizontalCenter: parent.horizontalCenter
               anchors.verticalCenter: parent.verticalCenter
               width: parent.width
@@ -143,7 +143,7 @@ Variants {
 
             // Bottom section (right widgets)
             Column {
-              spacing: Style.marginXS * root.scaling
+              spacing: Style.marginS * root.scaling
               anchors.horizontalCenter: parent.horizontalCenter
               anchors.bottom: parent.bottom
               anchors.bottomMargin: Style.marginM * root.scaling

Modules/Bar/Widgets/ActiveWindow.qml

@@ -52,7 +52,7 @@ Item {
   readonly property real minWidth: Math.max(1, screen.width * 0.06)
   readonly property real maxWidth: minWidth * 2
 
-  implicitHeight: Math.round(Style.capsuleHeight * scaling)
+  implicitHeight: (barPosition === "left" || barPosition === "right") ? calculatedVerticalHeight() : Math.round(Style.barHeight * scaling)
   implicitWidth: (barPosition === "left" || barPosition === "right") ? Math.round(Style.baseWidgetSize * 0.8 * scaling) : (horizontalLayout.implicitWidth + Style.marginM * 2 * scaling)
 
   function getTitle() {
@@ -66,6 +66,21 @@ Item {
 
   visible: getTitle() !== ""
 
+  function calculatedVerticalHeight() {
+    // Base height for the background rectangle
+    let total = Math.round(Style.capsuleHeight * scaling)
+
+    // Add padding for the container margins
+    total += Style.marginM * scaling * 2 // Top and bottom margins
+
+    // Add space for icon if shown
+    if (showIcon) {
+      total += Style.fontSizeL * scaling * 1.2 + Style.marginS * scaling
+    }
+
+    return total
+  }
+
   function calculatedHorizontalWidth() {
     let total = Style.marginM * 2 * scaling // internal padding
 

Modules/Bar/Widgets/Clock.qml

@@ -33,23 +33,12 @@ Rectangle {
   readonly property bool use12h: widgetSettings.use12HourClock !== undefined ? widgetSettings.use12HourClock : widgetMetadata.use12HourClock
   readonly property bool reverseDayMonth: widgetSettings.reverseDayMonth !== undefined ? widgetSettings.reverseDayMonth : widgetMetadata.reverseDayMonth
   readonly property string displayFormat: widgetSettings.displayFormat !== undefined ? widgetSettings.displayFormat : widgetMetadata.displayFormat
-
+  
   // Use compact mode for vertical bars
   readonly property bool useCompactMode: barPosition === "left" || barPosition === "right"
 
   implicitWidth: useCompactMode ? Math.round(Style.capsuleHeight * scaling) : Math.round(layout.implicitWidth + Style.marginM * 2 * scaling)
   implicitHeight: useCompactMode ? Math.round(Style.capsuleHeight * 2.5 * scaling) : Math.round(Style.capsuleHeight * scaling)
-
-  // React to bar position changes
-  Connections {
-    target: BarService
-    function onBarPositionChanged(newPosition) {
-      root.barPosition = newPosition
-      // Force re-evaluation of implicitWidth and implicitHeight
-      root.implicitWidth = Qt.binding(() => useCompactMode ? Math.round(Style.capsuleHeight * scaling) : Math.round(layout.implicitWidth + Style.marginM * 2 * scaling))
-      root.implicitHeight = Qt.binding(() => useCompactMode ? Math.round(Style.capsuleHeight * 2.5 * scaling) : Math.round(Style.capsuleHeight * scaling))
-    }
-  }
   radius: Math.round(Style.radiusS * scaling)
   color: Color.mSurfaceVariant
 
@@ -75,20 +64,18 @@ Rectangle {
             if (useCompactMode) {
               // Compact mode: time section (first 2 lines)
               switch (index) {
-              case 0:
-                // Hours
-                if (use12h) {
-                  const hours = now.getHours()
-                  const displayHours = hours === 0 ? 12 : (hours > 12 ? hours - 12 : hours)
-                  return displayHours.toString().padStart(2, '0')
-                } else {
-                  return now.getHours().toString().padStart(2, '0')
-                }
-              case 1:
-                // Minutes
-                return now.getMinutes().toString().padStart(2, '0')
-              default:
-                return ""
+                case 0: // Hours
+                  if (use12h) {
+                    const hours = now.getHours()
+                    const displayHours = hours === 0 ? 12 : (hours > 12 ? hours - 12 : hours)
+                    return displayHours.toString().padStart(2, '0')
+                  } else {
+                    return now.getHours().toString().padStart(2, '0')
+                  }
+                case 1: // Minutes
+                  return now.getMinutes().toString().padStart(2, '0')
+                default:
+                  return ""
               }
             } else {
               // Normal mode: single line with time
@@ -165,14 +152,12 @@ Rectangle {
             if (useCompactMode) {
               // Compact mode: date section (last 2 lines)
               switch (index) {
-              case 0:
-                // Day
-                return now.getDate().toString().padStart(2, '0')
-              case 1:
-                // Month
-                return (now.getMonth() + 1).toString().padStart(2, '0')
-              default:
-                return ""
+                case 0: // Day
+                  return now.getDate().toString().padStart(2, '0')
+                case 1: // Month
+                  return (now.getMonth() + 1).toString().padStart(2, '0')
+                default:
+                  return ""
               }
             }
             return ""

Modules/Bar/Widgets/MediaMini.qml

@@ -58,6 +58,10 @@ Item {
     return MediaService.trackTitle + (MediaService.trackArtist !== "" ? ` - ${MediaService.trackArtist}` : "")
   }
 
+  function calculatedVerticalHeight() {
+    return Math.round(Style.baseWidgetSize * 0.8 * scaling)
+  }
+
   function calculatedHorizontalWidth() {
     let total = Style.marginM * 2 * scaling // internal padding
     if (showAlbumArt) {
@@ -70,7 +74,7 @@ Item {
     return total
   }
 
-  implicitHeight: Math.round(Style.capsuleHeight * scaling)
+  implicitHeight: (barPosition === "left" || barPosition === "right") ? calculatedVerticalHeight() : Math.round(Style.barHeight * scaling)
   implicitWidth: (barPosition === "left" || barPosition === "right") ? Math.round(Style.baseWidgetSize * 0.8 * scaling) : (rowLayout.implicitWidth + Style.marginM * 2 * scaling)
 
   visible: MediaService.currentPlayer !== null && MediaService.canPlay

Modules/Bar/Widgets/SystemMonitor.qml

@@ -51,7 +51,7 @@ Item {
   readonly property bool showNetworkStats: (widgetSettings.showNetworkStats !== undefined) ? widgetSettings.showNetworkStats : widgetMetadata.showNetworkStats
   readonly property bool showDiskUsage: (widgetSettings.showDiskUsage !== undefined) ? widgetSettings.showDiskUsage : widgetMetadata.showDiskUsage
 
-  implicitHeight: (barPosition === "left" || barPosition === "right") ? calculatedVerticalHeight() : Math.round(Style.capsuleHeight * scaling)
+  implicitHeight: (barPosition === "left" || barPosition === "right") ? calculatedVerticalHeight() : Math.round(Style.barHeight * scaling)
   implicitWidth: (barPosition === "left" || barPosition === "right") ? Math.round(Style.capsuleHeight * scaling) : (horizontalLayout.implicitWidth + Style.marginL * 2 * scaling)
 
   function calculatedVerticalHeight() {

Services/BarService.qml

@@ -7,8 +7,8 @@ import qs.Commons
 Singleton {
   id: root
 
-  // Bar position property - initialize safely
-  property string position: "top"
+  // Bar position property
+  property string position: Settings.data.bar.position
 
   // Signal emitted when bar position changes
   signal barPositionChanged(string newPosition)
@@ -17,7 +17,7 @@ Singleton {
   Connections {
     target: Settings
     function onDataChanged() {
-      if (Settings.data && Settings.data.bar && Settings.data.bar.position !== root.position) {
+      if (Settings.data.bar.position !== root.position) {
         root.position = Settings.data.bar.position
         root.barPositionChanged(root.position)
       }
@@ -31,15 +31,6 @@ Singleton {
 
   // Function to change bar position
   function setPosition(newPosition) {
-    if (Settings.data && Settings.data.bar) {
-      Settings.data.bar.position = newPosition
-    }
-  }
-
-  // Initialize position after component is completed
-  Component.onCompleted: {
-    if (Settings.data && Settings.data.bar) {
-      position = Settings.data.bar.position
-    }
+    Settings.data.bar.position = newPosition
   }
 }