After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 653646 - Add HTML Notification support
Add HTML Notification support
Status: RESOLVED DUPLICATE of bug 741295
Product: epiphany
Classification: Core
Component: General
unspecified
Other All
: Normal enhancement
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
: 642347 698961 738911 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-06-29 14:04 UTC by Alexandre Mazari
Modified: 2014-12-10 17:45 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add HTML Notification support (16.66 KB, patch)
2011-06-29 14:04 UTC, Alexandre Mazari
none Details | Review
Add HTML Notification support (18.63 KB, patch)
2011-06-30 14:03 UTC, Alexandre Mazari
none Details | Review
Add HTML Notification support (18.80 KB, patch)
2011-06-30 14:12 UTC, Alexandre Mazari
needs-work Details | Review
Chrome global notifications settings (10.71 KB, image/png)
2011-11-15 15:38 UTC, Alexandre Mazari
  Details
Chrome notification auth exception page (15.51 KB, image/png)
2011-11-15 15:38 UTC, Alexandre Mazari
  Details

Description Alexandre Mazari 2011-06-29 14:04:31 UTC
Provide a WebKitWebNotificationPresenter implementation using libnotify to present notifications to the user.
Comment 1 Alexandre Mazari 2011-06-29 14:04:34 UTC
Created attachment 190932 [details] [review]
Add HTML Notification support

Implement a WebKitNotificationPresenter using libnotify to present notifications to the user.
Comment 2 Alexandre Mazari 2011-06-29 14:06:25 UTC
TODO:
- share and persist authorizations/denials of origins
- add a global setting to ignore notifications requests
Comment 3 Alexandre Mazari 2011-06-30 14:03:30 UTC
Created attachment 191029 [details] [review]
Add HTML Notification support

Implement a WebKitNotificationPresenter using libnotify to present notifications to the user.
Comment 4 Alexandre Mazari 2011-06-30 14:12:21 UTC
Created attachment 191031 [details] [review]
Add HTML Notification support

Listen to WebKitWebView notifications related signals.
Create an info-bar for permission requests, and delegates actual notification displaying to the notifications daemon via libnotify.
The support can be enabled using the '--enable-notifications' configure flag.

This updated patch follows the signals approach as implemented in
https://bugs.webkit.org/show_bug.cgi?id=61140

What is missing:
- global setting to enable/disable notifications support
- persistence of user response (positive or negative)
- raising the origin ephyembed of a clicked notification
- ?
Comment 5 Alexandre Mazari 2011-11-15 15:37:39 UTC
This code still build and work with current webkit patches as seen in http://people.igalia.com/amazari/ephy-html-notifications.webm

I am currently wondering how to present the settings for this feature to the user.

A global switch, with 3 alternatives should controls the overhaul behaviour of the notification system:
- display all notification without user input == automatically accept all notifications permission requests
- never display notification == forbid any notification permission request
- ask the user whenever a unknown security origin (domain) request permission to display notifications.

In the later case, the permissions should be stored and made available to the user to modify/revoke accepted notifications permissions.

The global switch would naturally fit in Preferences>Confidentiality imho, but this tab is already quite crowded, so we need to figure out a way to pack it/relayout it.

The authorization exceptions list might fit as a new tab in 'Personal Data' dialog, I guess.

For reference I am attaching chrome (17) notification preferences pages


Before heading there, opinions are welcome, even more with the redesign coming.

Happy Coding.
Comment 6 Alexandre Mazari 2011-11-15 15:38:22 UTC
Created attachment 201448 [details]
Chrome global notifications settings
Comment 7 Alexandre Mazari 2011-11-15 15:38:58 UTC
Created attachment 201449 [details]
Chrome notification auth exception page
Comment 8 Xan Lopez 2012-07-31 20:57:52 UTC
Comment on attachment 191031 [details] [review]
Add HTML Notification support

Unfortunately the whole notification stuff in WebKit has changed a lot lately, so we'll need to re-take this at some point and adapt it.
Comment 9 Alexandre Franke 2013-05-31 10:49:18 UTC
*** Bug 698961 has been marked as a duplicate of this bug. ***
Comment 10 William Jon McCann 2013-05-31 13:39:00 UTC
*** Bug 642347 has been marked as a duplicate of this bug. ***
Comment 11 Paul Neulinger 2014-07-14 20:47:16 UTC
I think that the Notification API for desktop notifications (http://www.w3.org/TR/notifications/) is very important - if not crucial - for a browser that actively supports web applications. There are many use cases where desktop notifications come in handy and are essential for a true replacement of native applications like web interfaces for messaging/email services (new message), calendar/todo (reminder), streaming services (currently playing),... 

Many of them already implement the Notification API and many do not have a native alternative for gnome/linux systems. Would be cool...
Comment 12 Claudio Saavedra 2014-10-21 10:41:37 UTC
*** Bug 738911 has been marked as a duplicate of this bug. ***
Comment 13 Gustavo Noronha (kov) 2014-12-10 17:45:07 UTC

*** This bug has been marked as a duplicate of bug 741295 ***