I think this should be "" instead of "undefined" otherwise leaving a state might not reset stuff correctly.
===
18 + if (notificationList.currentIndex == index) {
and following lines...
You shouldn't reach out of scope here (Can break if someone. Please either user ListView.view instead of notificationList or move the logic out into the containing ListView.
Why can't this be a binding in the ListView? Not sure why this needs to be called every time a new delegate is created.
===
Not sure if related to this branch (and if its a real issue after all), but if I trigger sd-example-incoming-call.py and then decline the incoming call the notification doesn't go away (it works fine if I accept the call)
After that I added a sleep 1 in between the lines, it still causes issues and at some point an empty rectangle appears.
===
If the first notification is expanded, and a new one comes in at position 0, the new one shows up expanded and then quickly collapses. I think that might be related to this:
15 + var result = "undefined";
I think this should be "" instead of "undefined" otherwise leaving a state might not reset stuff correctly.
===
18 + if (notificationLi st.currentIndex == index) {
and following lines...
You shouldn't reach out of scope here (Can break if someone. Please either user ListView.view instead of notificationList or move the logic out into the containing ListView.
===
small coding style nitpick
70 anchors.fill: parent margins margins rightMargin: notification. margins
71 - anchors.margins: notification.
72 + anchors.leftMargin: notification.
73 + anchors.
anchors { fill: parent; ... }
===
117 + Component. onCompleted: { t.spacing = 0 t.spacing = units.gu(.5)
118 + if (fullscreen) {
119 + notificationLis
120 + } else {
121 + notificationLis
122 + }
123 + }
Why can't this be a binding in the ListView? Not sure why this needs to be called every time a new delegate is created.
===
Not sure if related to this branch (and if its a real issue after all), but if I trigger sd-example- incoming- call.py and then decline the incoming call the notification doesn't go away (it works fine if I accept the call)
===
I'm calling this in a script:
./sd-example- incoming- call-canceled. py & incoming- call.py & incoming- file.py & morning- alarm.py & much-body- text.py & password- entry.py & simunlock. py & user-auth. py & using-button- tint-hint. py &
./sd-example-
./sd-example-
./sd-example-
./sd-example-
./sd-example-
./sd-example-
./sd-example-
./sd-example-
=> unity crashes.
After that I added a sleep 1 in between the lines, it still causes issues and at some point an empty rectangle appears.
===
If the first notification is expanded, and a new one comes in at position 0, the new one shows up expanded and then quickly collapses. I think that might be related to this:
125 + ListView.onAdd: { st.currentIndex < 1 && notificationLis t.count > 1) { t.currentIndex = 1
126 + if (notificationLi
127 + notificationLis
128 + }
The problem here is that you only correct the currentIndex after everything is built up. If you make it a binding it should work better.
===
207 + wait(500);
Is that a leftover from testing?
===
tests look good. Maybe rename them to give it a more meaningful name? make testVisualQueue does not really tell its notifications.