Skip to content

refactor: simplify watched marking to swipe-past model#114

Merged
GraysonCAdams merged 2 commits intomainfrom
refactor/simplify-watched-marking
Mar 12, 2026
Merged

refactor: simplify watched marking to swipe-past model#114
GraysonCAdams merged 2 commits intomainfrom
refactor/simplify-watched-marking

Conversation

@GraysonCAdams
Copy link
Collaborator

Summary

  • Replace complex deferred/threshold watched logic with a simpler swipe-past model: clips are marked watched when the user swipes forward past them
  • ClipOverlay marks clips watched immediately on open
  • Remove deferWatched, deferFirstClip props and associated timers from ReelItem
  • Clean up console.logs and use onMount for feed initialization
  • Sort comments oldest-first (instead of newest) when hearts are equal

Test plan

  • Open feed, swipe through clips — verify clips are marked watched when swiped past (not on a timer)
  • Open a clip via ClipOverlay — verify it's marked watched immediately
  • Check Me page reel view — verify watched marking works on swipe
  • Verify comments sort: equal-heart comments appear oldest-first
  • Verify unwatched count decrements correctly as clips are swiped past

Replace the complex deferred/threshold watched logic with a simpler
approach: mark a clip watched when the user swipes past it. ClipOverlay
marks immediately on open. Removes deferWatched, deferFirstClip props
and associated timers from ReelItem. Cleans up console.logs and uses
onMount for feed initialization.
@GraysonCAdams GraysonCAdams enabled auto-merge (squash) March 12, 2026 03:19
@GraysonCAdams GraysonCAdams merged commit dfbc5fd into main Mar 12, 2026
20 checks passed
@GraysonCAdams GraysonCAdams deleted the refactor/simplify-watched-marking branch March 12, 2026 03:23
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