Skip to content

add download button loading and outcome states#57

Merged
JeremiahM37 merged 3 commits into
JeremiahM37:mainfrom
sgerner:codex/download-button-states
Jun 2, 2026
Merged

add download button loading and outcome states#57
JeremiahM37 merged 3 commits into
JeremiahM37:mainfrom
sgerner:codex/download-button-states

Conversation

@sgerner

@sgerner sgerner commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

This change makes the search-result download button provide immediate visual feedback while the initial network request is pending and after it resolves.

What changed:

  • Added a per-result loading spinner and disabled state while POST /api/download is in flight.
  • Added transient success and error button states after the request completes.
  • Added translations for the new button labels.
  • Extended the same spinner treatment to the downloads tab refresh, clear-completed, and per-job retry actions.

Why:

  • The download flow was otherwise silent during the initial request, which made the UI feel unresponsive.
  • A success or error state on the button gives immediate confirmation of the network result without relying only on toast notifications.

Impact:

  • Users can tell at a glance whether the request is still pending, succeeded, or failed.
  • The behavior is localized to the UI and does not change backend download handling.

Checks:

  • Parsed the inline browser scripts successfully.
  • Rebuilt the local librarr:prowlarr-audiobooks image.
  • Recreated the running librarr container.

@sgerner sgerner marked this pull request as ready for review June 1, 2026 01:08
Comment thread internal/organize/pipeline.go Fixed
Comment thread internal/organize/pipeline.go Fixed
Comment thread internal/organize/pipeline.go Fixed
Comment thread internal/organize/pipeline.go Fixed
Comment thread internal/organize/pipeline.go Fixed
@JeremiahM37 JeremiahM37 self-assigned this Jun 1, 2026

@JeremiahM37 JeremiahM37 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@JeremiahM37 JeremiahM37 assigned sgerner and unassigned JeremiahM37 Jun 1, 2026
@JeremiahM37 JeremiahM37 merged commit b7d076c into JeremiahM37:main Jun 2, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants