Skip to content

fix(docs-infra): align the page title with its edit action#69424

Merged
kirjs merged 1 commit into
angular:mainfrom
erkamyaman:docs-infra-align-page-title-edit-icon
Jun 18, 2026
Merged

fix(docs-infra): align the page title with its edit action#69424
kirjs merged 1 commit into
angular:mainfrom
erkamyaman:docs-infra-align-page-title-edit-icon

Conversation

@erkamyaman

Copy link
Copy Markdown
Contributor

The page title row (.docs-page-title) relied on the default flex alignment, so the edit icon next to the title did not line up with the title text. Add align-items: baseline so the icon sits on the title's baseline.

PR Checklist

Please check if 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... Please describe:

What is the current behavior?

The edit (pencil) icon next to a page title is not vertically aligned with the
title text, because .docs-page-title uses the default flex alignment.

image image

What is the new behavior?

The edit icon aligns to the title's baseline via align-items: baseline.

image image

Does this PR introduce a breaking change?

  • Yes
  • No

The page title row (`.docs-page-title`) relied on the default flex
alignment, so the edit icon next to the title did not line up with the
title text. Add `align-items: baseline` so the icon sits on the title's
baseline.
@pullapprove pullapprove Bot requested a review from kirjs June 18, 2026 12:37
@angular-robot angular-robot Bot added the area: docs-infra Angular.dev application and infrastructure label Jun 18, 2026
@ngbot ngbot Bot added this to the Backlog milestone Jun 18, 2026
@github-actions

Copy link
Copy Markdown

Deployed adev-preview for 3f5a086 to: https://ng-dev-previews-fw--pr-angular-angular-69424-adev-prev-8n96ifgl.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Jun 18, 2026
@JeanMeche JeanMeche removed the request for review from kirjs June 18, 2026 17:42
@kirjs kirjs merged commit e77a8a0 into angular:main Jun 18, 2026
27 checks passed
@kirjs

kirjs commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

This PR was merged into the repository. The changes were merged into the following branches:

@erkamyaman erkamyaman deleted the docs-infra-align-page-title-edit-icon branch June 18, 2026 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker adev: preview area: docs-infra Angular.dev application and infrastructure target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants