834393 - Don't allow an unusable popup notification

Status: NEW

(Toolkit :: PopupNotifications and Notification Bars, enhancement)

Description

[[e-mail]]

It's possible to create a popup notification with an invisible anchor or no anchor at all, in which case a default anchor is used instead, however it is impossible to retrieve such a notification once it has been dismissed.

The problem comes when you create a dismissed notification with such an anchor, since it then becomes completely unusable.

In bug 831365 this was worked around in the caller by checking whether the anchor was invisible before asking for the notification not to be shown automatically.

My suggestion is that popup notifications should always be shown if the anchor is null or invisible. (It might be worth writing a shared _isAnchorUsable method.)

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

How should the user reshow one of these notifications? What happens if there are multiple that want to be displayed at the same time?

The patch for bug 831365 should be backed out when a patch for this bug lands.

Comment 2

[Matthew N. [:MattN]]

Bulk move to Toolkit::Notifications and Alerts

Filter on notifications-and-alerts-component.