nw.js icon indicating copy to clipboard operation
nw.js copied to clipboard

Visual push Notification showing chrome icon

Open meraj321 opened this issue 4 years ago • 10 comments

NWJS Version : 0.45.0 Operating System : Win10

Expected behavior

Visual notification icon should be configurable, that is, it should be possible to enable or disable chrome logo on visual notification through package.json

Actual behavior

Visual notification is displaying chrome logo at left-top corner and custom (application's) logo on bottom-right corner.

How to reproduce

It can be easily reproduce by using build in notification APIs. I used sample code from https://github.com/nwjs/nw.js/wiki/Notification, and it is reproducible with that.

meraj321 avatar Apr 10 '20 06:04 meraj321

Hi,

This issue is still reproducible with nw.js version 49. It's displaying a built- in chrome logo along with an app icon in push notification.

Thanks, Meraj

meraj321 avatar Nov 11 '20 05:11 meraj321

notify

After opening all files that ship with NW.js (SDK and Normal) in Resource Hacker, none of them contained a Chromium logo as a replaceable icon.

Reproduction:

Can reproduce in:

Version Flavor OS Arch
0.42.5 SDK Windows 7 x64
0.49.2 Normal Windows 7 x64
0.49.2 SDK Windows 7 x64
0.49.2 SDK Ubuntu 17 x64

Cannot reproduce in:

Version Flavor OS Arch
0.14.7 SDK Windows 7 x64
0.33.4 SDK Windows 7 x64
0.49.2 SDK OSX 10.11 x64

TheJaredWilcurt avatar Nov 11 '20 14:11 TheJaredWilcurt

I think nw.dll is linked with binary format of chromium logo.

Thanks, Meraj

On Wed 11 Nov, 2020, 8:00 PM The Jared Wilcurt, [email protected] wrote:

Can reproduce in 0.49.2 (normal and SDK) (on Windows 7 x64).

[image: notify] https://user-images.githubusercontent.com/4629794/98823495-28521500-2400-11eb-80ae-1353bc214dd8.png

After opening all files that ship with NW.js (SDK and Normal) in Resource Hacker, none of them contained a Chromium logo as a replaceable icon.

Reproduction:

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nwjs/nw.js/issues/7435#issuecomment-725455053, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKVAJGRE4VXM265WAAQGS2LSPKNZTANCNFSM4MFIFP6A .

meraj321 avatar Nov 11 '20 17:11 meraj321

How to solve this problem?I'm using version 0.50.2

gouxiwen avatar Feb 20 '21 07:02 gouxiwen

https://developer.mozilla.org/en-US/docs/Web/API/Notification

just replace img by a blank image

const notification = new Notification('To do list', {
  body: text,
  icon: img
});

jonlepage avatar Feb 20 '21 11:02 jonlepage

just replace img by a blank image

image That doesn't solve my problem

gouxiwen avatar Feb 20 '21 12:02 gouxiwen

i dont know , is work on my side ! image

image

jonlepage avatar Feb 20 '21 13:02 jonlepage

i dont know , is work on my side ! image

image

I see. Using badge, you can change the default chrome icon,thanks!

gouxiwen avatar Feb 21 '21 01:02 gouxiwen

i dont know , is work on my side ! I see. Using badge, you can change the default chrome icon,thanks!

ya and the official documentation have weird description ! They maybe need change this , because it mean nothing ! image

jonlepage avatar Feb 21 '21 02:02 jonlepage

Still doesn't fixed, example for reproduce it:

chrome.notifications.create("xxx", { "title": "Bacon Ipsum", "type": "basic", "priority": 2, "contextMessage": "Bacon Ipsum", "requireInteraction": true, "iconUrl": "data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7", "message": "Bacon ipsum dolor amet short ribs cupim jerky, ribeye boudin shoulder salami fatback beef ribs turducken strip steak. Swine cow turkey picanha shankle spare ribs." }, () => { const err = chrome.runtime.lastError; console.log(err || true); })

panther7 avatar Apr 07 '21 10:04 panther7