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

Notifications aren't removed from the status bar once read in the UI #18159

Open
leo60228 opened this issue Apr 28, 2022 · 3 comments
Open

Notifications aren't removed from the status bar once read in the UI #18159

leo60228 opened this issue Apr 28, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@leo60228
Copy link
Contributor

Steps to reproduce the problem

  1. Receive a notification
  2. Don't interact with it from the dropdown
  3. Read (and dismiss) it from the website

Expected behaviour

Notification should disappear from status bar

Actual behaviour

Notification remains in status bar until dismissed again

Specifications

I'm using glitch-soc@bb12af7 (includes upstream 3.5.1), but I don't see any differences in the relevant areas from upstream.
I'm on Android 11 and using Firefox 100.0b7.

@leo60228 leo60228 added the bug Something isn't working label Apr 28, 2022
@ClearlyClaire
Copy link
Contributor

I'm not sure what you mean. How was it dismissed, and from where doesn't it get removed?

@leo60228
Copy link
Contributor Author

Completely forgot to answer this, sorry... Here's the specific order of events:

  1. Open Mastodon in a browser tab
  2. Allow notifications
  3. Switch to another application
  4. Receive a notification from Mastodon
  5. Without dismissing it from Android's notification drawer, return to the Mastodon tab
  6. Read the notification (so that the Mastodon UI considers the notification to no longer be unread)

After doing this, the notification is still in Android's notification drawer. I assume a similar issue exists on iOS and possibly some desktop platforms. I believe that it should be possible for Mastodon to solve this using Notification.close?

@ClearlyClaire
Copy link
Contributor

ClearlyClaire commented May 12, 2022

Alright! Yeah, we should probably do that. Currently, we do not hold any reference to the notifications once they are created, but maybe we can use something like https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/notifications/getAll (EDIT: that is a WebExtension API)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants