[Mac OS X] desktop notification: better use "Alerts" if user interaction is required

[michaelstingl]

ownCloud client shows "Banners" to notify users about sync activities.

Banners appear in the upper-right corner of your screen, without interrupting what you're doing. Banners appear for a short period of time, then slide off screen to the right.

Sometimes, user interaction is required (expiring SAML session). With 2.1.1, those notifications also are displayed as "Banner" and disappear after 2 seconds:

I would suggest to display trigger so called "Alerts" instead for such notifications.

Alerts are important announcements that appear similar to banners. Alerts remain on your screen until you take action. Many alerts include buttons you can use to interact with them without having to open the app that generated them.

More information about notifications on Mac OS X:
https://support.apple.com/en-us/HT204079

@MTRichards @dragotin What do you think?

00004467

[MTRichards]

Interested in @danimo too.

In my experience, there is not universal agreement on this. It tends to be a highly personal and subjective answer based on the users and what they like.

[michaelstingl]

@MTRichards Yes, you are right. That's why users can go to "System Preferences" > "Notifications" and configure for each software individually to display alerts as "Banner".

[MTRichards]

interesting. Still looking at @danimo and @dragotin for thoughts, plus @jancborchardt for UI.

[guruz]

Which build did you try? AFAIK recent 2.1.1 nightlies should not Show this notification at all but Pop up the auth Dialog
8486a2f

@jturcotte please correct me if needed

[jturcotte]

In 2.1.1 it should pop together with an shibboleth authentication window, only if the token expired. The notification shouldn't be shown alone on its own like it does in 2.1.0 anymore.

[MorrisJobke]

In 2.1.1 it should pop together with an shibboleth authentication window, only if the token expired. The notification shouldn't be shown alone on its own like it does in 2.1.0 anymore.

This was communicated -> Ticket could be closed.

[guruz]

I'm all for putting this in 2.4 (if there is one, else 3.0)

FYI @pmaier1 ?

[michaelstingl]

@dragotin did some investigation about advanced Qt libs that could help here. I remember there have been issues with licence incompatbility.

[guruz]

@michaelstingl We already do things on OS X natively, don't really need a library for that. https://github.com/owncloud/client/blame/b7809ded23fdba6aa50b6d834b5bd5fc16865eea/src/gui/systray.mm

[pmaier1]

I'm all for putting this in 2.4 (if there is one, else 3.0)
FYI @pmaier1 ?

Agreed that this change is needed, agreed to postpone!

[guruz]

Related #4697