Skip to content

fix(layout): increase gap between sidebar and content#1154

Open
LeoMcA wants to merge 1 commit intomainfrom
sidebar-gap
Open

fix(layout): increase gap between sidebar and content#1154
LeoMcA wants to merge 1 commit intomainfrom
sidebar-gap

Conversation

@LeoMcA
Copy link
Member

@LeoMcA LeoMcA commented Dec 10, 2025

A fix for #690 (comment), increasing the gap between content and the sidebar content by ~1rem to 3rem

Also fixes #593: left sidebar no longer overlaps with header anchor hashes

@LeoMcA LeoMcA requested a review from a team as a code owner December 10, 2025 18:45
@LeoMcA LeoMcA requested review from argl and caugner and removed request for argl December 10, 2025 18:45
@github-actions
Copy link
Contributor

github-actions bot commented Dec 10, 2025

85dd580 was deployed to: https://fred-pr1154.review.mdn.allizom.net/

@LeoMcA
Copy link
Member Author

LeoMcA commented Dec 10, 2025

@Rumyra this PR increases the gap between sidebar and content by 1rem:

before:

Screen Shot 2025-12-10 at 18 57 22

vs after:

Screen Shot 2025-12-10 at 18 57 26

I'd recommend viewing the images fullscreen, or following the links to the pages themselves. It's easy to increase the gap by even more if you want: modify --layout-sidebar-gap in devtools and see what looks good.

@LeoMcA LeoMcA changed the base branch from main to layout-consolidation January 30, 2026 09:51
Base automatically changed from layout-consolidation to main February 3, 2026 17:57
@LeoMcA LeoMcA added the needs content decision This issue needs a decision from the MDN Content Team before it can proceed. label Feb 3, 2026
@caugner
Copy link
Contributor

caugner commented Feb 3, 2026

@mdn/content-team When you have a moment, can you please review this change (e.g. compare the review deployment to prod)?

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

Labels

needs content decision This issue needs a decision from the MDN Content Team before it can proceed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Left sidebar scrollbar overlaps headings anchor icon

2 participants