Skip to content

Conversation

@brandonkachen
Copy link
Collaborator

@brandonkachen brandonkachen commented Sep 25, 2025

Change Section component to use minHeight instead of height when fullViewport is true. This ensures the hero section maintains minimum viewport height but can expand to accommodate IDE demo content, preventing cutoff on smaller screens.

🤖 Generated with Codebuff

…ent IDE demo cutoff

Change Section component to use minHeight instead of height when fullViewport is true.
This ensures the hero section maintains minimum viewport height but can expand
to accommodate IDE demo content, preventing cutoff on smaller screens.

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>
@brandonkachen brandonkachen temporarily deployed to fix/hero-section-content-based-sizing - web PR #315 September 25, 2025 23:03 — with Render Destroyed
@brandonkachen brandonkachen changed the title [render preview] fix(web): change hero section from fixed height to min-height to prev… [render preview] fix(web): change hero section to fit better on smaller screens Sep 25, 2025
brandonkachen and others added 2 commits September 25, 2025 16:26
…E demo cutoff

Removed overflow-hidden from Section component and added bottom padding
to hero container to ensure IDE demo content is fully visible.

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>
Reduce hero section bottom padding from 10rem to 5rem on mobile
while keeping 10rem on desktop for better mobile experience.

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>
@brandonkachen brandonkachen temporarily deployed to fix/hero-section-content-based-sizing - web PR #315 September 26, 2025 00:08 — with Render Destroyed
@brandonkachen brandonkachen merged commit dfa47f6 into main Sep 26, 2025
14 checks passed
@brandonkachen brandonkachen deleted the fix/hero-section-content-based-sizing branch September 26, 2025 00: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