Skip to content

fix(left-sidebar): unindent nested details#1312

Draft
caugner wants to merge 1 commit intomainfrom
1311-unindent-nested-sidebar-nodes
Draft

fix(left-sidebar): unindent nested details#1312
caugner wants to merge 1 commit intomainfrom
1311-unindent-nested-sidebar-nodes

Conversation

@caugner
Copy link
Contributor

@caugner caugner commented Feb 17, 2026

Description

Updates the left-sidebar layout, unindenting nested details:

  • If the details element has nested details, unindent (only) the summary element.
  • If the details element has no nested details, unindent the whole details element.

Motivation

  1. Avoid confusion.
  2. Save horizontal space (possibly less line breaks in nested sidebars).

Additional details

Sidebar Before After
CSS image image
WebExt image image

Related issues and pull requests

Fixes #1311.

@caugner caugner requested a review from a team as a code owner February 17, 2026 11:59
@caugner caugner requested a review from LeoMcA February 17, 2026 11:59
@github-actions
Copy link
Contributor

01dd10c was deployed to: https://fred-pr1312.review.mdn.allizom.net/

@caugner caugner marked this pull request as draft February 17, 2026 12:49
@caugner caugner marked this pull request as ready for review February 17, 2026 14:52
@caugner
Copy link
Contributor Author

caugner commented Feb 17, 2026

At first, I thought the new behavior for CSS properties is confusing, but it's actually consistent with the current behavior at the top level, so I think it's fine.

@caugner caugner added the needs content decision This issue needs a decision from the MDN Content Team before it can proceed. label Feb 17, 2026
@LeoMcA
Copy link
Member

LeoMcA commented Feb 23, 2026

I think this is overly complex, and piles on the edge-cases - a path we already know to be dangerous with the sidebars.

I'd first take a step back and re-evaluate how we consider the sidebar hierarchy, which I've attempted to do in #1311 (comment). We can then revisit and fix some remaining edge-cases with the not-yet-baseline-green :has selector - it shouldn't be the basis of our whole hierocratical display.

@LeoMcA LeoMcA marked this pull request as draft February 23, 2026 18:28
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.

Nested sidebar items show visually misleading hierarchy

2 participants