Browser extension |notifications| API needs support for buttons in notifications

NEW
Unassigned

Status

P3
normal
4 years ago
a month ago

People

(Reporter: billm, Unassigned)

Tracking

(Depends on: 1 bug, Blocks: 3 bugs, {dev-doc-needed})

34 Branch
dev-doc-needed
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [notifications]triaged)

This will require some toolkit support.
Priority: -- → P1
Component: Extension Compatibility → WebExtensions
Product: Firefox → Toolkit
Keywords: dev-doc-needed

Updated

3 years ago
Whiteboard: [notifications]

Updated

3 years ago
Depends on: 1213455

Updated

3 years ago
Flags: blocking-webextensions-

Updated

3 years ago
Whiteboard: [notifications] → [notifications]triaged
Blocks: 1213455
No longer depends on: 1213455
No longer blocks: 1256627
See Also: → bug 1274559

Comment 1

3 years ago
I'm not sure this is a P1 (it was filed as such a year ago), Bob did we have some tracking bugs for what needs to happen to allow this to work?
Flags: needinfo?(bob.silverberg)
Depends on: 1286536
This is required for Chrome parity, but I agree that it may not be a P1. Native work is required to update nsIAlertsService to support the buttons. It doesn't look like a bug has been opened for that, so I created bug 1286536 and have marked it as blocking this bug. There are a few other things that we don't seem to support from chrome.notifications, such as notifications with multiple items, notifications which report progress, and the `requireInteraction` option [1]. I'm not sure where we stand on implementing those or not. Kris, do you have any thoughts on those?

[1] https://developer.chrome.com/extensions/notifications#type-NotificationOptions
Depends on: 1225110
Flags: needinfo?(bob.silverberg)
Flags: needinfo?(kmaglione+bmo)

Comment 3

3 years ago
Thanks for adding the blocking bug. Agree, its not a P1 so dropping.
Priority: P1 → P3
I agree that it doesn't seem like a huge priority at the moment, given the amount of platform work required. Let's look at it again in the next quarter, or when bug 1225110 is fixed.
Flags: needinfo?(kmaglione+bmo)

Updated

2 years ago
Component: WebExtensions: Untriaged → WebExtensions: Frontend
Flags: blocking-webextensions-
Duplicate of this bug: 1348726

Comment 6

2 years ago
Any tentative date to fix this bug ? I'll plan my web extension accordingly.
(In reply to sankar88.aj from comment #6)
> Any tentative date to fix this bug ? I'll plan my web extension accordingly.

No, the code that generates notifications (independent of webextensions) doesn't yet have support for buttons, so that needs to happen before the corresponding webextensions apis get implemented.  You can follow the dependent bug linked above, but it doesn't appear to be a high priority for anybody at the moment.

Comment 8

2 years ago
Now that all legacy add-ons are disabled in the Firefox nightly, maybe this should have a higher priority.

Adblock Plus (WebExtensions) falls back to Window.confirm if notifications with buttons are not supported, but that won't work for WebExtensions either because bug #1203394. There is currently no fallback available.

Comment 10

2 years ago
Thanks, windows.create might work as a fallback. It's still not available on Android though.

Updated

2 years ago
Blocks: 467520, 1226547

Comment 11

a year ago
With the release of Firefox Quantum my Gmail extension could definitely use a notification button to quickly delete incoming emails, just as it does with my the Chrome version of the extension.

Comment 12

10 months ago
Half a year later, still an issue. This doesn't seem to be blocked by anything.

Updated

8 months ago
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.