Skip to content

boosted 3cols#1784

Merged
simo6529 merged 2 commits intomainfrom
simo6529/boosted-3cols
Jan 21, 2026
Merged

boosted 3cols#1784
simo6529 merged 2 commits intomainfrom
simo6529/boosted-3cols

Conversation

@simo6529
Copy link
Copy Markdown
Collaborator

@simo6529 simo6529 commented Jan 21, 2026

Summary by CodeRabbit

  • Refactor
    • Replaced Masonry-based layouts with responsive CSS grid layouts, improving consistency across mobile, tablet, and desktop views.
    • Converted horizontal scrolling containers to responsive grid layouts for better usability across all screen sizes.
    • Streamlined internal component interactions to reduce complexity and improve system reliability.

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

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

coderabbitai Bot commented Jan 21, 2026

📝 Walkthrough

Walkthrough

This PR refactors layout components by replacing Masonry-based and horizontal scroll layouts with responsive CSS grids, while simultaneously removing the onNavigate callback prop from wave creator preview components, delegating navigation to native Link behavior.

Changes

Cohort / File(s) Summary
Layout Grid Conversion
components/home/boosted/BoostedSection.tsx, components/home/explore-waves/ExploreWavesSection.tsx
Replaced Masonry and horizontal scroll layouts with responsive CSS grid (1–3 columns). Removed multi-breakpoint complexity (isXl, isMd); simplified column calculation logic. Updated dependencies and removed margin utilities on grid items.
Navigation Prop Cleanup
components/waves/drops/WaveCreatorPreviewItem.tsx, components/waves/drops/WaveCreatorPreviewModalContent.tsx
Removed onNavigate prop from WaveCreatorPreviewItemProps and all usages. Component now relies solely on Link href for navigation instead of callback-based routing.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Suggested reviewers

  • ragnep

Poem

🐰 Masonry walls come tumbling down,
Grid lines paint the responsive town,
Callbacks fade like morning dew,
Links now lead where they're due!

🚥 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 'boosted 3cols' refers to the primary change visible in the main file BoostedSection.tsx, where the layout is refactored to use responsive CSS grid with 3 columns on large screens.

✏️ 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

sonarqubecloud Bot commented Jan 21, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
1 Accepted issue

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@prxt6529 prxt6529 self-requested a review January 21, 2026 18:23
@simo6529 simo6529 merged commit 9ff66f1 into main Jan 21, 2026
7 checks passed
@simo6529 simo6529 deleted the simo6529/boosted-3cols branch January 21, 2026 18:24
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