Skip to content

wip#1822

Merged
simo6529 merged 1 commit intomainfrom
scroll-to-undread-button-rework
Jan 26, 2026
Merged

wip#1822
simo6529 merged 1 commit intomainfrom
scroll-to-undread-button-rework

Conversation

@simo6529
Copy link
Copy Markdown
Collaborator

@simo6529 simo6529 commented Jan 26, 2026

Summary by CodeRabbit

  • New Features

    • Enhanced unread message indicator with automatic dismissal and improved visibility detection.
  • Improvements

    • Refined styling and layout adjustments for avatar and messaging components.
    • Improved responsive design for message notifications.
  • Updates

    • Security policy enhancements.

✏️ Tip: You can customize this high-level summary in your review settings.

Signed-off-by: Simo <simo@6529.io>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 26, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Multiple wave-related UI components are updated: WaveAvatar styling is refactored with WavePicture component removal and class adjustments; SingleWaveDropChat consolidates imports and restructures JSX layout; WaveDropsScrollToUnreadButton introduces visibility-based state tracking and auto-dismiss logic; and Content-Security-Policy removes worker-src directive.

Changes

Cohort / File(s) Summary
Wave Avatar Styling
components/brain/left-sidebar/web/WebBrainLeftSidebarWave/subcomponents/WaveAvatar.tsx
Removed WavePicture component import and rendering; adjusted opacity and ring classes for active/inactive states; refactored badge and drop icon positioning with updated Tailwind classNames.
Wave Drop Chat Refactoring
components/waves/drop/SingleWaveDropChat.tsx
Consolidated type and value imports from same module; reformatted paddingBottom ternary and style props to multi-line format; reorganized wrapper and nested container div structure while preserving functional content.
Unread Divider Behavior
components/waves/drops/WaveDropsScrollToUnreadButton.tsx
Added state tracking (hasSeenDivider, wasInViewRef, hasAutoDismissedRef, lastSerialNoRef) for visibility-based auto-dismiss; integrated MutationObserver and scroll listeners; enhanced dismissal event handling with suppressClick flag; updated arrow direction logic and responsive text wrapping.
Security Configuration
config/securityHeaders.ts
Removed worker-src directive from Content-Security-Policy string in createSecurityHeaders function.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~30 minutes

Possibly related PRs

  • Dismiss new messages button #1815: Modifies WaveDropsScrollToUnreadButton to add onDismiss prop and swipe-to-dismiss interaction, building on the visibility-based auto-dismiss logic introduced here.
  • Unread divider fix #1688: Updates unread-divider handling and WaveDropsScrollToUnreadButton visibility/state logic for the same unread divider wiring functionality.
  • Clean nextjs config file #1717: Modifies config/securityHeaders.ts to manage Content-Security-Policy directives alongside this worker-src removal change.

Suggested reviewers

  • ragnep
  • prxt6529

Poem

🐰 Avatars now shine without their pictures bright,
Drops reorganize with layout new in sight,
Dividers watch and wait with state so keen,
Auto-dismiss their magic, clean and lean—
Security tightens, worker-src takes flight!

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@simo6529 simo6529 merged commit cd3bec6 into main Jan 26, 2026
4 of 5 checks passed
@simo6529 simo6529 deleted the scroll-to-undread-button-rework branch January 26, 2026 09:37
@sonarqubecloud
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot mentioned this pull request Feb 2, 2026
@coderabbitai coderabbitai Bot mentioned this pull request Apr 9, 2026
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