✨ Nextcloud 16 polishing

[nickvergessen]

• Add eslint with the same rules as in other apps

---

• Dismissing older notification also hides every newer notification (only until refresh)
• Notifications window should have min-height (same as when it’s empty)
• Needs short compress animation for dismissing for proper feedback

[nickvergessen]

Needs short compress animation for dismissing for proper feedback

Can a @nextcloud/javascript do this or tell me how? I guess it goes here and fade is not enough/working?

notifications/src/components/Notification.vue

Line 213 in c349dcc

// this._$el.fadeOut(OC.menuSpeed);

Dismissing older notification also hides every newer notification (only until refresh)

Although the logic is the same as in all other apps I used Vue in until now, index is undefined in

notifications/src/App.vue

Line 118 in c349dcc

onRemove: function(index) {

removing always the first item, so when you delete teh second one of two, the list shows nothing anymore.

[MorrisJobke]

removing always the first item, so when you delete teh second one of two, the list shows nothing anymore.

FYI: Having 3 notifications then deleting the second results in a list with only the third one.

[nickvergessen]

@skjnldsv

• Fixed position of empty content
• Transition in chromium on last removal

[skjnldsv]

DOne