Skip to content

fix: set min-height on docs main content for sticky layout - #27

Merged
loadinglucian merged 1 commit into
mainfrom
fix/docs-main-content-min-height
Feb 19, 2026
Merged

fix: set min-height on docs main content for sticky layout#27
loadinglucian merged 1 commit into
mainfrom
fix/docs-main-content-min-height

Conversation

@loadinglucian

Copy link
Copy Markdown
Owner

Summary

  • Adds min-h-[calc(100vh-var(--docs-sticky-top)-2rem)] to the docs <main> element so the content area always fills the viewport below the sticky header, preventing sidebar collapse on short pages.

@coderabbitai

coderabbitai Bot commented Feb 19, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

A minimum height calculation class is added to the main content element in the docs layout component. This ensures proper vertical spacing when a sticky header is present. It's a single-line CSS class modification affecting layout behavior.

Changes

Cohort / File(s) Summary
Docs Layout Update
resources/views/components/layouts/docs.blade.php
Added min-h-[calc(100vh-var(--docs-sticky-top)-2rem)] CSS class to main content element for proper vertical space allocation with sticky headers.

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically summarizes the main change: adding min-height styling to the docs main content element for sticky layout support.
Description check ✅ Passed The description is directly related to the changeset, providing context about the min-height addition and its purpose in preventing sidebar collapse on short pages.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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

@loadinglucian
loadinglucian merged commit 39d1202 into main Feb 19, 2026
7 checks passed
@loadinglucian
loadinglucian deleted the fix/docs-main-content-min-height branch February 19, 2026 09:15
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