Skip to content

[UI] fix(docs): prevent sidebar overlap when browser zoom is increased#1142

Merged
rishiraj38 merged 3 commits into
layer5io:masterfrom
KumarNirupam1:fix/1140-docs-layout-zoom-overlap
Jul 12, 2026
Merged

[UI] fix(docs): prevent sidebar overlap when browser zoom is increased#1142
rishiraj38 merged 3 commits into
layer5io:masterfrom
KumarNirupam1:fix/1140-docs-layout-zoom-overlap

Conversation

@KumarNirupam1

Copy link
Copy Markdown
Contributor

Notes for Reviewers

This PR fixes #1140

When the browser zoom level is increased, the left navigation sidebar overlaps the main documentation content, hiding parts of the page and making it difficult to read and navigate.

This PR applies a minimal CSS-only fix in the existing resizable layout overrides:

  • Added min-width: 0 to .row > main so the main content can shrink correctly inside the flex row instead of overflowing underneath the sidebar.
  • Extended the existing sidebar stacking breakpoint from 768px to 991.98px (Bootstrap lg) so zoomed and narrower viewports use the stacked layout before the fixed-width sidebar can overlap main content.

Screenshoot

Screenshot 2026-07-09 222154

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Kumar Nirupam <kumar.nirupam24@gmail.com>
@welcome

welcome Bot commented Jul 9, 2026

Copy link
Copy Markdown

Yay, your first pull request! 👍 A contributor will be by to give feedback soon. In the meantime, please review the Layer5 Community Welcome Guide and sure to join the community Slack.
Be sure to double-check that you have signed your commits. Here are instructions for making signing an implicit activity while peforming a commit.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces styling adjustments to _resizable-bootstrap-overrides.scss by adding min-width: 0 to .row > main and updating the media query breakpoint from 768px to 991.98px. The reviewer suggests using Bootstrap's built-in responsive mixin @include media-breakpoint-down(lg) instead of hardcoding the max-width value to improve maintainability and ensure consistency with Bootstrap's grid breakpoints.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread assets/scss/_resizable-bootstrap-overrides.scss Outdated
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.6.3
Preview removed because the pull request was closed.
2026-07-12 12:56 UTC

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Kumar Nirupam <145343368+KumarNirupam1@users.noreply.github.com>
@KumarNirupam1

KumarNirupam1 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

@Sbragul26 please review 🙂

@rishiraj38 rishiraj38 merged commit ba017f0 into layer5io:master Jul 12, 2026
4 checks passed
@welcome

welcome Bot commented Jul 12, 2026

Copy link
Copy Markdown

        Thank you for contributing to the Layer5 community! 🎉 \ \ Congrats! \ \         ⭐ Please leave a star on the project. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UI] Docs layout breaks when browser zoom is increased

2 participants