Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notification with tag works only once #11189

Closed
manix opened this issue Nov 20, 2017 · 7 comments · Fixed by #18401
Closed

Notification with tag works only once #11189

manix opened this issue Nov 20, 2017 · 7 comments · Fixed by #18401

Comments

@manix
Copy link

manix commented Nov 20, 2017

After creating a notification and passing a tag, then subsequent notifications with the same tag don't show up for the entire lifespan of the app.

  • Electron version: 1.8.1
  • Operating system: Windows 7 x64

Expected behavior

New notifications should open after the last one has closed.

Actual behavior

Nothing happens

How to reproduce

$ git clone https://github.com/electron/electron-quick-start .
$ npm install
$ npm start

Then in the app open dev tools and paste this in the console:

new Notification("test", { tag: "test" });

Wait for the notification to disappear and paste it again, observe nothingness.

@welcome
Copy link

welcome bot commented Nov 20, 2017

👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.

To help make it easier for us to investigate your issue, please follow the contributing guidelines.

@MarshallOfSound
Copy link
Member

For clarity this is not a 1.8.x regression, it only occurs on Windows 7 and it's due to how we attempt to de-duplicate notification objects based on there tag in our windows 7 notification presenter

@gasi-signal
Copy link

gasi-signal commented May 4, 2018

We ran into this as well on the Signal Desktop app.

Repro

windows-7-notifications-static-tag

Code

let counter = 0;
let intervalId = setInterval(
  () => new Notification(counter++, { tag: "static-tag" }),
  1000
);

gasi-signal added a commit to signalapp/Signal-Desktop that referenced this issue May 4, 2018
gasi-signal added a commit to signalapp/Signal-Desktop that referenced this issue May 4, 2018
gasi-signal added a commit to signalapp/Signal-Desktop that referenced this issue May 4, 2018
Grouping of notifications is not supported on Windows 7 due this bug:
electron/electron#11189

*   [x] Disable notification grouping (using `tag`) on Windows 7.
*   [x] Log notification grouping support.
*   [x] **Infrastructure:** Use 2-space indentation for all files
        (better integration with Prettier.)
@sofianguy
Copy link
Contributor

Thank you for taking the time to report this issue and helping to make Electron better.

The version of Electron you reported this on has been superseded by newer releases.

If you're still experiencing this issue in Electron v2.0.x or later, please add a comment specifying the version you're testing with and any other new information that a maintainer trying to reproduce the issue should know.

I'm setting the blocked/need-info label for the above reasons.

Thanks in advance! Your help is appreciated.

@sofianguy sofianguy added the blocked/need-info ❌ Cannot proceed without more information label Jan 7, 2019
@sofianguy
Copy link
Contributor

Thank you for your issue!

We haven't gotten a response to our questions in our comment above. With only the information that is currently in the issue, we don't have enough information to take action. I'm going to close this but don't hesitate to reach out if you have or find the answers we need, we'll be happy to reopen the issue.

@r0stig
Copy link
Contributor

r0stig commented Apr 17, 2019

Hello,

I just want to point out that I still get this bug on electron 4.1.4.

I'm able to reproduce the bug with the the code @gasi-signal posted and a clean electron-quick-start-repo.

Edit: When I actively close the notification new notifications will always popup. But when I wait for the notification to auto-dismiss next notifications with the same tag won't appear. So I guess the auto-close don't release it's resources?

@r0stig
Copy link
Contributor

r0stig commented Apr 25, 2019

Just letting you know the bug is still present in 5.0.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants