notifs: fix opening anim when spam notifs

No open/close state, just animate height

modules/drawers/Drawers.qml

@@ -105,7 +105,6 @@ Variants {
                 id: visibilities
 
                 property bool osd
-                property bool notifications: Notifs.popups.length > 0
                 property bool session
                 property bool launcher
                 property bool dashboard

modules/drawers/Panels.qml

@@ -40,8 +40,6 @@ Item {
     Notifications.Wrapper {
         id: notifications
 
-        visibility: root.visibilities.notifications
-
         anchors.top: parent.top
         anchors.right: parent.right
     }

modules/notifications/Content.qml

@@ -10,34 +10,19 @@ Item {
 
     readonly property int padding: Appearance.padding.large
 
+    anchors.top: parent.top
     anchors.bottom: parent.bottom
     anchors.right: parent.right
 
-    implicitWidth: NotifsConfig.sizes.width + root.padding * 2
+    implicitWidth: NotifsConfig.sizes.width + padding * 2
-    implicitHeight: list.implicitHeight + root.padding * 2
-
-    ClippingWrapperRectangle {
-        anchors.top: parent.top
-        anchors.left: parent.left
-        anchors.right: parent.right
-        anchors.margins: root.padding
-
-        color: "transparent"
-        radius: Appearance.rounding.normal
-
-        ListView {
-            id: list
-
-            model: ScriptModel {
-                values: [...Notifs.popups].reverse()
-            }
-
-            anchors.fill: parent
-
     implicitHeight: {
-                let height = (count - 1) * spacing;
+        const count = list.count;
+        if (count === 0)
+            return 0;
+
+        let height = (count - 1) * list.spacing;
         for (let i = 0; i < count; i++)
-                    height += itemAtIndex(i)?.nonAnimHeight ?? 0;
+            height += list.itemAtIndex(i)?.nonAnimHeight ?? 0;
 
         const screen = QsWindow.window?.screen;
         const visibilities = Visibilities.screens[screen];
@@ -56,9 +41,25 @@ Item {
             }
         }
 
-                return Math.max(61, Math.min((screen?.height ?? 0) - root.padding * 2 - BorderConfig.thickness * 2, height));
+        return Math.min((screen?.height ?? 0) - BorderConfig.thickness * 2, height + padding * 2);
     }
 
+    ClippingWrapperRectangle {
+        anchors.fill: parent
+        anchors.margins: root.padding
+
+        color: "transparent"
+        radius: Appearance.rounding.normal
+
+        ListView {
+            id: list
+
+            model: ScriptModel {
+                values: [...Notifs.popups].reverse()
+            }
+
+            anchors.fill: parent
+
             orientation: Qt.Vertical
             spacing: Appearance.spacing.smaller
             cacheBuffer: QsWindow.window?.screen.height ?? 0
@@ -92,12 +93,12 @@ Item {
                     property: "y"
                 }
             }
+        }
+    }
 
     Behavior on implicitHeight {
         Anim {}
     }
-        }
-    }
 
     component Anim: NumberAnimation {
         duration: Appearance.anim.durations.expressiveDefaultSpatial

modules/notifications/Wrapper.qml

@@ -1,35 +1,13 @@
 import "root:/config"
-import Quickshell
 import QtQuick
 
 Item {
     id: root
 
-    required property bool visibility
-
     visible: height > 0
-    implicitHeight: 0
+    implicitHeight: content.implicitHeight
     implicitWidth: content.implicitWidth + BorderConfig.rounding
 
-    states: State {
-        name: "visible"
-        when: root.visibility
-
-        PropertyChanges {
-            root.implicitHeight: content.implicitHeight
-        }
-    }
-
-    transitions: Transition {
-        NumberAnimation {
-            target: root
-            property: "implicitHeight"
-            duration: Appearance.anim.durations.expressiveDefaultSpatial
-            easing.type: Easing.BezierSpline
-            easing.bezierCurve: Appearance.anim.curves.expressiveDefaultSpatial
-        }
-    }
-
     Content {
         id: content
     }