Make sure notifications are always on top

[call-me-matt]

When triggering a OC.Notification.showTemporary, I realized it is not always on top?

Expected behavior:
Notification has a higher z-index than Modal

[ChristophWurst]

@nextcloud/designers is there a sane value for the z-index? Would there ever be anything above the toast?

[skjnldsv]

Well, we really have no global list of zindex, so it's kind of everyone is doing their own sh** 🙈

[jancborchardt]

The other issue is that even if it has a proper z-index, it would then overlay the modal controls. So we would possibly have to move it to the left/bottom too?

[skjnldsv]

well, this is a live temporary heads up, I think it's fair to have it on top?

[jancborchardt]

Sure – we can always see for further enhancements anyway. :)

[susnux]

Not sure how bad this is performance wise, but theoretically:

findZIndex() {
    // Get all elements
    return [...document.querySelectorAll('body *')]
        // use getComputedStyle to also include zIndex from stylesheets
        .map(el => parseFloat(window.getComputedStyle(el).zIndex))
        // Reduce to get only the highest value
        .reduce((current, z) => z > current ? z : current, 1)
}