nextcloud / nextcloud/notifications

Dismiss notification should hide directly

Open
#786 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug design feature: frontend 🖌️ help wanted
Dominant language
JavaScript
Stars
153
Forks
67
Avg merge
15h 3m
Merged PRs (30d)
96

Description

Steps
  1. Have some notifications present under the bell icon
  2. Open notifications dropdown
  3. Click the cross icon to dismiss a notification
Expected result

Notification disappears directly.

Actual result

Delay of 1-2 seconds until it disappears.

For a better UX, we should hide the notification right away even if the XHR call is still ongoing. Worst case we can still re-display it in case of an error.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing the notification dropdown's dismiss handler and the XHR call triggered by the cross icon. Verify that the notification is hidden immediately, and determine how an error could restore it. Done means the dropdown updates without the current 1–2 second delay while failed requests remain recoverable.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.