Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

HTML5 push notifications not working in Windows 10 #1556

Closed
bbondy opened this issue May 4, 2016 · 13 comments
Closed

HTML5 push notifications not working in Windows 10 #1556

bbondy opened this issue May 4, 2016 · 13 comments

Comments

@bbondy
Copy link
Member

bbondy commented May 4, 2016

Doesn't seem to work in Chrome either for me. Also previous versions doesn't work.

@bbondy
Copy link
Member Author

bbondy commented May 4, 2016

This was tested on Windows 7 by the way, it might work on win8 and win10.

@diracdeltas
Copy link
Member

Both notifications from the page and notifications from the browser context?

@bbondy
Copy link
Member Author

bbondy commented May 4, 2016

both I suspect just win7 but I haven't tried on win10 yet.
The permissions all work correctly before the no-op notification though.

@bbondy bbondy changed the title HTML5 push notifications not working in Windows HTML5 push notifications not working in Windows 7 May 5, 2016
@bbondy
Copy link
Member Author

bbondy commented May 5, 2016

Tested in Windws 10 x64 and from the content dev tools new Notification('hi') works.
However the browser level ones in the browser console have no effect.

This differs than OS X where it works with both consoles.

@bbondy bbondy changed the title HTML5 push notifications not working in Windows 7 HTML5 push notifications not working in Windows 10 May 5, 2016
@jonathansampson
Copy link
Collaborator

I just tested http://www.bennish.net/web-notifications.html on Windows 10 (10586.318), and it appears to be working correctly.

Brave: 0.10.0
Electron: 0.37.8
libchromiumcontent: 50.0.2661.102
V8: 5.0.71.48
Node.js: 5.10.0
Update channel: dev

brave notifications

@bsclifton
Copy link
Member

bsclifton commented May 22, 2016

@confirmed on Windows 10 also! It works and looks great :) And because of the recent chromium updates, as @jonathansampson noticed yesterday, it uses the Windows 10 noticiation center! I'll open an issue for that, since the app name it shows could be better

I wonder if this can be retested on Windows 7 / 8? (although, the issue name specifically calls out Windows 10)

edit:
I'm using Windows 10 Home (64 bit, build 10586.318)

@luixxiul
Copy link
Contributor

luixxiul commented May 23, 2016

On my Windows 10 (build 14342) the test causes a crash. I don't know why.

STR

  1. Allow Scripts
  2. click "Show"
  3. click "Allow"

It doesn't crash if you click "remember this decision" and "Authorize" at first and then click "Show", but in this case no notification is displayed :/

@bsclifton
Copy link
Member

Thanks for trying and also capturing the version info. I updated my post above to capture what I have (which does work). I'm guessing you're in the Windows Insider program (if so, which track)?

@luixxiul
Copy link
Contributor

luixxiul commented May 23, 2016

I'm guessing you're in the Windows Insider program (if so, which track)?

Yes, Slow ring. I'm not sure the crash also happens on the Fast ring version.

This might be related with #1726

@bridiver
Copy link
Collaborator

I retested on win7 and still no notifications

@luixxiul
Copy link
Contributor

@bridiver I will check later on win10

@diracdeltas
Copy link
Member

may be related: #2297

@luixxiul luixxiul added this to the Backlog milestone Jan 5, 2017
@luixxiul luixxiul added the bug label Jun 4, 2017
@bbondy bbondy removed this from the Backlog milestone Oct 19, 2017
@luixxiul
Copy link
Contributor

luixxiul commented Nov 7, 2017

This has not caused a crash for a very long time. Closing.

@luixxiul luixxiul closed this as completed Nov 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants