Differential D26452

Handle web push notifications with Falkon notification system
ClosedPublic
Actions

Authored by SGOrava on Jan 6 2020, 9:33 AM.

Details

Reviewers
None
Group Reviewers
Falkon
Commits
R875:2a0f0924362a: Handle web push notifications with Falkon notification system
Summary

Add basic support for web notifications with icon, title and body text.

FEATURE: 409045

Test Plan

Fire some notifications from webpage

Diff Detail

Repository
R875 Falkon
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
SGOrava created this revision.Jan 6 2020, 9:33 AM
Restricted Application added a project: Falkon. · View Herald TranscriptJan 6 2020, 9:33 AM
SGOrava requested review of this revision.Jan 6 2020, 9:33 AM
SGOrava added a comment.Jan 20 2020, 2:43 PM

If there aren't any objections I will land this patch tonight.
It provides the same basic functionality as QtWebEngine default bubble.

This revision was not accepted when it landed; it landed in state Needs Review.Jan 20 2020, 7:40 PM
Closed by commit R875:2a0f0924362a: Handle web push notifications with Falkon notification system (authored by SGOrava). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/lib/app/mainapplication.cpp (14 lines)

Diff 73978

View Options

src/lib/app/mainapplication.cpp

Loading...