Skip to content

fix: share auto-submit, favorites reel back-button & auto-scroll#48

Merged
GraysonCAdams merged 2 commits intomainfrom
fix/share-auto-submit-and-favorites-reel
Mar 3, 2026
Merged

fix: share auto-submit, favorites reel back-button & auto-scroll#48
GraysonCAdams merged 2 commits intomainfrom
fix/share-auto-submit-and-favorites-reel

Conversation

@GraysonCAdams
Copy link
Copy Markdown
Collaborator

Summary

  • Share page auto-submit: URL is now submitted automatically when valid and the platform is allowed — no manual tap needed. Shows a "Adding to feed…" loading state, and a dedicated error state with "Try again" on failure. Removes the old fromShortcut branching logic since auto-submit supersedes it.
  • Favorites reel back-button: Defer onReelPopState handler by one tick so SvelteKit can update page.state before we inspect it. This prevents the reel from closing when a bottom sheet on top (comments, viewers, etc.) is dismissed via the back gesture.
  • Favorites auto-scroll disabled: autoScroll was being passed from user prefs into the favorites reel — hardcoded to false since auto-advance doesn't make sense for a hand-curated list.

Test plan

  • Share a supported URL from another app (Android share sheet / iOS Shortcut) — confirm it auto-submits immediately with loading indicator
  • Share an unsupported/failing URL — confirm error state appears with "Try again" button
  • Open a clip in favorites reel, open comments sheet, dismiss via back gesture — confirm reel stays open
  • Open a clip in favorites reel, press the hardware back button — confirm reel closes and returns to grid
  • Verify favorites reel does not auto-advance to the next clip

- Auto-submit when URL is valid and platform is allowed (no tap needed)
- Show "Adding to feed…" loading state while submitting
- Show dedicated error state with "Try again" button on failure
- Remove fromShortcut branch — auto-submit supersedes the shortcut flow
- Fix .err CSS alias for error icon wrap styling
- Defer onReelPopState so SvelteKit updates page.state before we check it;
  prevents reel from closing when a sheet on top dismisses via back button
- Disable autoScroll in favorites reel (was incorrectly inherited from prefs)
@GraysonCAdams GraysonCAdams merged commit 7efc60a into main Mar 3, 2026
17 checks passed
@GraysonCAdams GraysonCAdams deleted the fix/share-auto-submit-and-favorites-reel branch March 3, 2026 01:27
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