Skip to content

Move header loading animation to CLI update checks - #14

Merged
brianlovin merged 1 commit into
mainfrom
brianlovin/header-loading-fix
Jan 26, 2026
Merged

Move header loading animation to CLI update checks#14
brianlovin merged 1 commit into
mainfrom
brianlovin/header-loading-fix

Conversation

@brianlovin

Copy link
Copy Markdown
Owner

Summary

  • Header loading spinner now only shows when checking for CLI updates
  • Removed redundant header loading from story fetching (full-screen loading bar already exists)
  • Header loading is now exclusively used during background update checks

Test plan

  • Run the app and verify no loading indicator in header during initial story fetch
  • The full-screen loading bar should appear instead
  • Header loading spinner should appear briefly when checking for updates in the background

The header loading spinner should only appear when checking for CLI updates, not when fetching stories. Story fetching already has a full-screen loading bar, so the header loading was redundant. This change makes the header loading animation exclusive to the background update check in index.ts.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@brianlovin
brianlovin merged commit bb1f9bc into main Jan 26, 2026
1 check passed
@brianlovin
brianlovin deleted the brianlovin/header-loading-fix branch January 26, 2026 05:24
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.

1 participant