Skip to content

refactor: Mobile layout api reference#67690

Open
aparzi wants to merge 1 commit intoangular:mainfrom
aparzi:fix-issue-67650
Open

refactor: Mobile layout api reference#67690
aparzi wants to merge 1 commit intoangular:mainfrom
aparzi:fix-issue-67650

Conversation

@aparzi
Copy link
Contributor

@aparzi aparzi commented Mar 14, 2026

Fix mobile layout shift in API reference

fixes #67650

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:

Issue Number: #67650

What is the new behavior?

Fix mobile layout shift in API reference

Does this PR introduce a breaking change?

  • Yes
  • No

@pullapprove pullapprove bot requested a review from alan-agius4 March 14, 2026 21:08
@JeanMeche JeanMeche requested review from JeanMeche and removed request for alan-agius4 March 14, 2026 22:54
@aparzi aparzi changed the title fix(adev): Mobile layout api reference fix: Mobile layout api reference Mar 14, 2026
@aparzi
Copy link
Contributor Author

aparzi commented Mar 14, 2026

Fixed lint error

@github-actions
Copy link

github-actions bot commented Mar 15, 2026 •
edited
Loading

Deployed adev-preview for 83d6bb9 to: https://ng-dev-previews-fw--pr-angular-angular-67690-adev-prev-aa9nqlrn.web.app

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

@aparzi aparzi changed the title fix: Mobile layout api reference refactor: Mobile layout api reference Mar 15, 2026
@aparzi
Copy link
Contributor Author

aparzi commented Mar 15, 2026

I fixed error commit

@JeanMeche JeanMeche mentioned this pull request Mar 16, 2026
Closed
@aparzi aparzi requested a review from JeanMeche March 16, 2026 17:59
@SkyZeroZx
Copy link
Contributor

SkyZeroZx commented Mar 16, 2026

It looks good.
Although I’d just have a small opinion: I think it would be good if the input (search), select (package), and the chips were aligned so it looks visually consistent.

image

@aparzi
Copy link
Contributor Author

aparzi commented Mar 16, 2026

@SkyZeroZx
I'm agree, I have update the PR

JeanMeche
JeanMeche reviewed Mar 19, 2026
View reviewed changes
@leonsenft leonsenft added target: patch This PR is targeted for the next patch release area: docs-infra Angular.dev application and infrastructure labels Mar 19, 2026
@ngbot ngbot bot added this to the Backlog milestone Mar 19, 2026
@SkyZeroZx
Copy link
Contributor

SkyZeroZx commented Mar 20, 2026 •
edited
Loading

IMHO, we can also adjust the width of the filter and package select boxes to make them look more aligned.

And maybe add padding to the chips so we have the same spacing between the elements

image

@aparzi
Copy link
Contributor Author

aparzi commented Mar 23, 2026

@SkyZeroZx @JeanMeche
I updated PR

@JeanMeche
Copy link
Member

JeanMeche commented Mar 23, 2026

image

The regular layout doesn't look right imho.

@aparzi
refactor: Mobile layout api reference
684397f 
Fix mobile layout shift in API reference

fix angular#67650
@aparzi
Copy link
Contributor Author

aparzi commented Mar 23, 2026 •
edited
Loading

image The regular layout doesn't look right imho.

I update the PR. Now in standard mode the behaviour is similar at original

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

Reviewers

@JeanMeche JeanMeche JeanMeche left review comments

Assignees

No one assigned

Labels

adev: preview area: docs-infra Angular.dev application and infrastructure target: patch This PR is targeted for the next patch release

Projects

None yet

Milestone

Backlog

Development

Successfully merging this pull request may close these issues.

Mobile layout shift in API Reference

4 participants

@aparzi @SkyZeroZx @JeanMeche @leonsenft