Skip to content

fix(docs-infra): prevent tab labels from being clipped#69639

Open
erkamyaman wants to merge 1 commit into
angular:mainfrom
erkamyaman:fix-docs-infra-tab-label-clipping
Open

fix(docs-infra): prevent tab labels from being clipped#69639
erkamyaman wants to merge 1 commit into
angular:mainfrom
erkamyaman:fix-docs-infra-tab-label-clipping

Conversation

@erkamyaman

Copy link
Copy Markdown
Contributor

The active docs tab applied a 2px bottom border that inactive tabs lacked, which shrank the active label's content box and clipped its descenders. It also set line-height: 1.5 only on the active label, so switching tabs nudged the text. Reserve the border as transparent on every tab and recolor it on the active one, and share the label line-height, so the letters are no longer cut and the label stays put when switching.

PR Checklist

Please check that your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.dev application / infrastructure changes
  • Other

What is the current behavior?

Screen.Recording.2026-07-03.at.22.26.35.mov

What is the new behavior?

Screen.Recording.2026-07-03.at.22.27.04.mov

Does this PR introduce a breaking change?

  • Yes
  • No

The active docs tab applied a 2px bottom border that inactive tabs lacked,
which shrank the active label's content box and clipped its descenders. It
also set `line-height: 1.5` only on the active label, so switching tabs
nudged the text. Reserve the border as transparent on every tab and recolor
it on the active one, and share the label `line-height`, so the letters are
no longer cut and the label stays put when switching.
@pullapprove pullapprove Bot requested a review from josephperrott July 3, 2026 19:34
@angular-robot angular-robot Bot added the area: docs-infra Angular.dev application and infrastructure label Jul 3, 2026
@ngbot ngbot Bot added this to the Backlog milestone Jul 3, 2026
@erkamyaman

Copy link
Copy Markdown
Contributor Author

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

Labels

area: docs-infra Angular.dev application and infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant