Skip to content

[Bug] Show refresh notice after using the R shortcut#4395

Open
Anthomos wants to merge 1 commit into
miniflux:mainfrom
Anthomos:fix-refresh-shortcut-toast
Open

[Bug] Show refresh notice after using the R shortcut#4395
Anthomos wants to merge 1 commit into
miniflux:mainfrom
Anthomos:fix-refresh-shortcut-toast

Conversation

@Anthomos

@Anthomos Anthomos commented Jun 9, 2026

Copy link
Copy Markdown

Summary

  • keep the keyboard shortcut refresh path on the current page instead of consuming the server flash message through fetch() redirects
  • surface the same background refresh success text as a client-side toast for shortcut-triggered refreshes
  • preserve redirect handling for auth/session flows

Why

Fixes #4358. The R shortcut refreshed feeds correctly, but its fetch() + reload path swallowed the flash message that the menu form submission relies on, so users got no visible confirmation.

Verification

  • go test ./...

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Inconsistent visual feedback between "Refresh all feeds" menu and the R keyboard shortcut

1 participant