From 67f0c482b3372656b4e31cad01fc88e67a2e0d80 Mon Sep 17 00:00:00 2001 From: LemmyCook Date: Wed, 27 Aug 2025 10:23:43 -0400 Subject: [PATCH] ActiveWindow+MediaMini: minor adjustments to width and spacing. - also removed double fallback icon for media mini when no artwork available --- Modules/Bar/Widgets/ActiveWindow.qml | 9 +++++---- Modules/Bar/Widgets/MediaMini.qml | 16 +++------------- 2 files changed, 8 insertions(+), 17 deletions(-) diff --git a/Modules/Bar/Widgets/ActiveWindow.qml b/Modules/Bar/Widgets/ActiveWindow.qml index 9fa84da..ed4613d 100644 --- a/Modules/Bar/Widgets/ActiveWindow.qml +++ b/Modules/Bar/Widgets/ActiveWindow.qml @@ -14,6 +14,7 @@ Row { property real scaling: ScalingService.scale(screen) // Title stays collapsed by default; expands only on hover property bool showingFullTitle: false + readonly property int minWidth: 120 anchors.verticalCenter: parent.verticalCenter spacing: Style.marginS * scaling @@ -63,7 +64,7 @@ Row { Row { id: row anchors.verticalCenter: parent.verticalCenter - spacing: Style.marginXS * scaling + spacing: Style.marginS * scaling // Window icon Item { @@ -85,10 +86,10 @@ Row { NText { id: titleText - // Fix collapsed width to 150px to avoid layout shifts with neighbors + // Fix collapsed width to 120px to avoid layout shifts with neighbors // Expand up to 400px on hover - width: mouseArea.containsMouse ? Math.min(fullTitleMetrics.contentWidth, 400 * scaling) : 150 * scaling - horizontalAlignment: mouseArea.containsMouse ? Text.AlignLeft : Text.AlignHCenter + width: mouseArea.containsMouse ? Math.min(Math.max(minWidth * scaling, fullTitleMetrics.contentWidth), 400 * scaling) : minWidth * scaling + horizontalAlignment: Text.AlignLeft text: getTitle() font.pointSize: Style.fontSizeS * scaling font.weight: Style.fontWeightMedium diff --git a/Modules/Bar/Widgets/MediaMini.qml b/Modules/Bar/Widgets/MediaMini.qml index a154f12..744f6f8 100644 --- a/Modules/Bar/Widgets/MediaMini.qml +++ b/Modules/Bar/Widgets/MediaMini.qml @@ -97,7 +97,7 @@ Row { Row { id: row anchors.verticalCenter: parent.verticalCenter - spacing: Style.marginXS * scaling + spacing: Style.marginS * scaling z: 1 // Above the visualizer NIcon { @@ -132,16 +132,6 @@ Row { borderWidth: 0 border.color: Color.transparent } - - // Fallback icon when no album art available - NIcon { - id: windowIconFallback - text: MediaService.isPlaying ? "pause" : "play_arrow" - font.pointSize: Style.fontSizeL * scaling - verticalAlignment: Text.AlignVCenter - anchors.verticalCenter: parent.verticalCenter - visible: getTitle() !== "" && !trackArt.visible - } } } @@ -149,10 +139,10 @@ Row { id: titleText // If hovered or just switched window, show up to 400 pixels - // If not hovered show up to 150 pixels + // If not hovered show up to 120 pixels width: (mouseArea.containsMouse) ? Math.min(fullTitleMetrics.contentWidth, 400 * scaling) : Math.min(fullTitleMetrics.contentWidth, - 150 * scaling) + 120 * scaling) text: getTitle() font.pointSize: Style.fontSizeS * scaling font.weight: Style.fontWeightMedium