Skip to content

feat: UI polish — reaction pills, swipe bounds, settings cleanup#133

Merged
GraysonCAdams merged 6 commits intomainfrom
feat/ui-polish-apr-2
Apr 2, 2026
Merged

feat: UI polish — reaction pills, swipe bounds, settings cleanup#133
GraysonCAdams merged 6 commits intomainfrom
feat/ui-polish-apr-2

Conversation

@GraysonCAdams
Copy link
Copy Markdown
Collaborator

Summary

  • Reaction pills: Replace vertical per-user reaction rows in comments with compact horizontal emoji pills (grouped by emoji + count). Tap to reveal popover with avatars showing who reacted.
  • Swipe bounds: Remove rubber-band animation when swiping past For Us (left) or Replay (right) tab edges.
  • Settings cleanup: Remove redundant profile header (avatar, username, phone, group pill) from settings — already on Me page.
  • Other: Cookie secure flag fix for local dev, activity feed extracted to dedicated route, assorted reaction/UI fixes.

Test plan

  • Open comments on a clip with reactions — verify pills show horizontally with correct counts
  • Tap a reaction pill — verify popover shows usernames with avatars
  • Tap elsewhere to dismiss popover
  • Swipe left on For Us tab — verify no movement/animation
  • Swipe right on Replay tab — verify no movement/animation
  • Open settings — verify no avatar/username/phone section at top
  • Verify Me page still has full profile editing
  • Test in both light and dark mode

Without this, cookies are rejected over plain HTTP during local dev,
breaking auth and theme persistence.
- Deduplicate reaction events to latest per user in comments API
- Debounce reaction notifications to avoid spam
- Support delete mode in mark-read endpoint
- Block scroll while reaction picker is open
- Increase progress bar touch target and show thumb on mobile
- Allow reply button visibility to be controlled by parent
- Add data-comment-id attribute for scroll targeting
- Fix holdFired reset in ActionSidebar
- Add onclose callback to BaseSheet
- Prevent textarea overflow in CommentInput
Extract activity/notifications from bottom sheet overlay to its own
/activity page with proper navigation tab.
Replace vertical per-user reaction rows with grouped horizontal pills
showing emoji + count. Tap a pill to reveal a popover with who reacted,
styled to match the viewers sheet.
Stop rubber-band effect when swiping left on For Us or right on Replay
since there's nothing beyond those edges.
Avatar, username, phone, and group pill are already shown on the Me
page — no need to duplicate them on settings.
@GraysonCAdams GraysonCAdams merged commit 664eefa into main Apr 2, 2026
38 of 47 checks passed
@GraysonCAdams GraysonCAdams deleted the feat/ui-polish-apr-2 branch April 2, 2026 23:26
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.

2 participants