git.iko.soy/unity8
on xenial help sign in
more fixes for fullscreen stuff

e.g. when pushed down by volume notification or when dismissing the unlock first and then doing it later from the indicator

commit5592a0750fccd0909ff8893b98dfabdd9df45410 authorLukáš Tinkl <lukas.tinkl@canonical.com> dateTue 2 Aug 2016, 13:33 (10 years ago) parentc550de2
2 files changed+9−8
compare with xenial
unifiedsplit
qml/Notifications/Notification.qml+2−8 qml/Notifications/Notifications.qml+7
qml/Notifications/Notification.qml+2−8
@@ -98,13 +98,6 @@ StyledItem {
9898 }
9999 }
100100
101− Binding {
102− target: ListView.view
103− property: "topmostIsFullscreen"
104− value: fullscreen
105− when: index == 0
106− }
107−
108101 function closeNotification() {
109102 if (index === ListView.view.currentIndex) { // reset to get the 1st snap decision expanded
110103 ListView.view.currentIndex = -1;
@@ -130,6 +123,7 @@ StyledItem {
130123 if (fullscreen) {
131124 notification.notification.urgency = Notification.Critical;
132125 }
126+ ListView.view.topmostIsFullscreen = fullscreen;
133127 }
134128
135129 Behavior on implicitHeight {
@@ -227,7 +221,7 @@ StyledItem {
227221 if (notification.type === Notification.Interactive) {
228222 notification.notification.invokeAction(actionRepeater.itemAt(0).actionId)
229223 } else {
230− notificationList.currentIndex = index;
224+ notification.ListView.view.currentIndex = index;
231225 }
232226 }
233227 onReleased: {
qml/Notifications/Notifications.qml+7
@@ -65,6 +65,13 @@ ListView {
6565 hasMouse: notificationList.hasMouse
6666 background: notificationList.background
6767
68+ Component.onCompleted: topmostIsFullscreen = false; // async, the factory loader will set fullscreen to true later
69+
70+ property int theIndex: index
71+ onTheIndexChanged: {
72+ ListView.view.topmostIsFullscreen = fullscreen; // when we get pushed down by e.g. volume notification
73+ }
74+
6875 // make sure there's no opacity-difference between the several
6976 // elements in a notification
7077 // FIXME: disabled all transitions because of LP: #1354406 workaround