Bug 388086 - showPassiveNotification() should be able to have a close button
Summary: showPassiveNotification() should be able to have a close button
Status: RESOLVED NOT A BUG
Alias: None
Product: frameworks-kirigami
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.41.0
Platform: Other Linux
: NOR wishlist
Target Milestone: Not decided
Assignee: Marco Martin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-12-20 18:23 UTC by Nate Graham
Modified: 2019-05-02 22:21 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2017-12-20 18:23:10 UTC
Kirigami's showPassiveNotification should have an easy facility to add a close button to the notification.

Right now, there is no easy way to do this, which is problematic for the following use cases:
- Short text with a long timer; the message annoyingly sticks around far after the user has finished reading it
- Long text with any length timer: the message disappears before the user has finished reading it

If these notifications could be given a close button, then developers could use a very long timeout, confident that the user could dismiss the message at their leisure once they've finished reading it. Right now they can't easily do that, so the message always sticks around for too long, or disappears too early.
Comment 1 Nate Graham 2017-12-20 18:27:59 UTC
It's true that you can click or touch the notification to dismiss it, but that's not obvious, and there's no reason you'd even try to do that on the desktop (in Desktop UIs, generally things don't disappear when you click on them). I didn't find out that this was possible until I started reading Kirigami's source code.
Comment 2 Nate Graham 2019-05-02 22:21:57 UTC
Nah. The real problem is that Discover is misusing these to display important information and should use actual dialogs and inlineMessages instead. See Bug 403791.