Comment 11 for bug 1295762

Revision history for this message
Lars Karlitski (larsu) wrote :

> I disagree. The "x-canonical-snap-decisions-timeout" is intentionally used in
> favour of the Freedesktop timeout feature, to indicate very clearly, that is
> is only applicable to snap-decisions notifications.

I think that's unnecessary for the same reason I thought introducing hints for
snap decision actions was unnecessary. It's unlikely that developers have a
problem understanding that the timeout only applies to a specific kind of
notification. They already need to know about the different kinds of
notifications and what they can do with them.

Adding additional hints only increases the complexity of the API and makes it
harder to switch between different kinds of notifications.

> Using the regular parameter of the Notify() method - and only applying it
> snap-decisioin notifications - would be misleading, because the intention of
> that timeout parameter is to be affecting all kinds of notifications.

We already go against the intention of that parameter by ignoring it
completely. In fact, we go against the freedesktop specification, which states
that notification servers need to honor this parameter.