Skip to content

Improve vertical alignment of site items and fix floating page aside #3560

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

zenoachtig
Copy link
Contributor

The vertical alignment of items on the site had grown inconsistent. This PR standardises the height again, and simplies the layout logic of the PageAside menu, especially in its floating mode (only used on OpenAPI pages).

Demo

Vertical alignment

Cleaned up the vertical alignment of the TOC, page header incl. page actions dropdown, and the page sidebar.

Before → After (ignore the header logo size, that's because of local vs prod image hosting)

Screenshot 2025-08-12 at 20 39 27 Screenshot 2025-08-12 at 20 39 30

PageAside menu

The page's aside menu has improved sticky behaviour and better scroll containers, along with better vertical alignment with the rest of the site.

Before → After (ignore the header logo size, that's because of local vs prod image hosting)

Screenshot 2025-08-12 at 20 35 48 Screenshot 2025-08-12 at 20 35 52

Floating PageAside

Gave slightly more horizontal room and improves the scrolling and layout of the floating menu.

Before → After (ignore the header logo size, that's because of local vs prod image hosting)

Screenshot 2025-08-12 at 20 35 24 Screenshot 2025-08-12 at 20 35 31

Floating PageAside offset bug

Fix a bug where the menu offset would not take into account the presence of an announcement banner

Before → After (ignore the header logo size, that's because of local vs prod image hosting)

Screenshot 2025-08-12 at 20 48 22 Screenshot 2025-08-12 at 20 48 40

Copy link

linear bot commented Aug 12, 2025

Copy link

changeset-bot bot commented Aug 12, 2025

🦋 Changeset detected

Latest commit: cc4e427

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
gitbook Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

argos-ci bot commented Aug 12, 2025

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
customers-v2 (Inspect) 👍 Changes approved 168 changed Aug 13, 2025, 12:26 PM
v2-cloudflare (Inspect) 👍 Changes approved 684 changed Aug 13, 2025, 12:35 PM
v2-vercel (Inspect) 👍 Changes approved 691 changed Aug 13, 2025, 12:29 PM

@zenoachtig zenoachtig merged commit bcfa8d8 into main Aug 13, 2025
16 checks passed
@zenoachtig zenoachtig deleted the zeno/rnd-7873-alignment-between-on-this-page-and-ask-dropdowns-is branch August 13, 2025 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants