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

Ability to dismiss notifications #5585

Closed
pancake-tongs opened this issue Apr 7, 2023 · 2 comments
Closed

Ability to dismiss notifications #5585

pancake-tongs opened this issue Apr 7, 2023 · 2 comments
Assignees
Labels
enhancement enhancement of a already implemented feature/code

Comments

@pancake-tongs
Copy link

How to use GitHub

  • Please use the 👍 reaction to show that you want to have the same feature implemented.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Feature description

The Nextcloud desktop client shows several kinds of notifications in the pop-up window that is displayed when a user clicks on the system tray icon. These can be about any number of things, though the ones that I most often see are related to recent updates to Nextcloud applied server-side. I'm running nextcloud server as a snap, though this isn't directly relevant. It does, however, emit a notification each time there is an update.

See screenshot: https://imgur.com/e2LWnrA

As you can see, the notifications pile up, and cannot be dismissed. Nextcloud desktop client does not show an 'X' button, and in fact doesn't seem to respond to right or left clicks at all on notification entries. Sometimes, I want to check to make sure a recent file has been synced, and I have to scroll through dozens of notifications to reach the first meaningful status messages about recent files. To clear out these notifications, I have to log in to my nextcloud account in a browser, which is an unintuitive and undesirable UX.

I'd like to suggest that a mechanism to dismiss / delete notifications be integrated into the desktop client.

@pancake-tongs pancake-tongs added the enhancement enhancement of a already implemented feature/code label Apr 7, 2023
camilasan pushed a commit that referenced this issue May 12, 2023
- Fix for #5606 and #5585.
- That is how it is done in the server and other clients.
- Update tests.

Signed-off-by: Camila <[email protected]>
@camilasan camilasan self-assigned this May 12, 2023
camilasan pushed a commit that referenced this issue May 15, 2023
- Fix for #5606 and #5585.
- That is how it is done in the server and other clients.
- Update tests.

Signed-off-by: Camila <[email protected]>
camilasan pushed a commit that referenced this issue May 15, 2023
- Fix for #5606 and #5585.
- That is how it is done in the server and other clients.
- Update tests.

Signed-off-by: Camila <[email protected]>
camilasan pushed a commit that referenced this issue May 16, 2023
- Fix for #5606 and #5585.
- That is how it is done in the server and other clients.
- Update tests.

Signed-off-by: Camila <[email protected]>
mgallien pushed a commit that referenced this issue May 16, 2023
- Fix for #5606 and #5585.
- That is how it is done in the server and other clients.
- Update tests.

Signed-off-by: Camila <[email protected]>
@jospoortvliet
Copy link
Member

This should be resolved with the work above, thanks for reporting!

@pancake-tongs
Copy link
Author

This issue has been subject to a regression and should be reopened. Please see: c975d40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement enhancement of a already implemented feature/code
Projects
None yet
Development

No branches or pull requests

3 participants