Skip to content

add gallery icon to mobile#1790

Merged
simo6529 merged 1 commit intomainfrom
simo6529/chat-gallery-toggle
Jan 22, 2026
Merged

add gallery icon to mobile#1790
simo6529 merged 1 commit intomainfrom
simo6529/chat-gallery-toggle

Conversation

@simo6529
Copy link
Copy Markdown
Collaborator

@simo6529 simo6529 commented Jan 22, 2026

Summary by CodeRabbit

  • New Features

    • Wave view mode functionality with a toggle button in the header to switch between gallery and chat modes
    • Profile picture display integrated into the application header
    • Enhanced header title formatting for improved readability
  • Style

    • Loading indicators in the gallery now display as static elements instead of animated spinners

✏️ 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 22, 2026

📝 Walkthrough

Walkthrough

The changes add wave view mode functionality and a gallery/chat toggle button to the application header with profile picture rendering support, while also removing spinner animations from gallery loading indicators.

Changes

Cohort / File(s) Summary
Wave View Mode & Gallery Toggle
components/header/AppHeader.tsx
Added wave view mode context integration with useWave and useWaveViewMode hooks, profile picture rendering logic, and a new conditional gallery/chat toggle button next to the title. Header title logic now uses last path segment for display. Avatar rendering replaced with pfpElement.
Gallery Loading States
components/waves/gallery/WaveGallery.tsx
Removed animate-spin class from two loading indicators, converting animated spinners to static circular loaders in both initial and "loading more" states.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

Suggested reviewers

  • ragnep
  • prxt6529

Poem

🐰 A toggle so fine in the header so grand,
Wave modes and pfps, a coordinated brand!
No more spinning spinners, just circles so still,
Our gallery blooms with a smoother good will!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'add gallery icon to mobile' refers to a real aspect of the changeset (the new gallery/chat toggle button UI), but it does not capture the main changes which include wave view mode functionality, header title logic updates, and PFP rendering logic.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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.

@sonarqubecloud
Copy link
Copy Markdown

@simo6529 simo6529 merged commit e08b795 into main Jan 22, 2026
7 checks passed
@simo6529 simo6529 deleted the simo6529/chat-gallery-toggle branch January 22, 2026 11:38
@coderabbitai coderabbitai Bot mentioned this pull request Mar 3, 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