Skip to content

landing ui cleanup, a11y improvements to tab navigation - #679

Merged
dcramer merged 11 commits into
getsentry:mainfrom
PickleNik:nikita/landing-cleanup
Dec 15, 2025
Merged

dcramer merged 11 commits into
getsentry:mainfrom
PickleNik:nikita/landing-cleanup

Conversation

@PickleNik

@PickleNik PickleNik commented Dec 12, 2025 •

Copy link
Copy Markdown
Member

before
Screenshot 2025-12-12 at 5 20 58 PM

after
Screenshot 2025-12-12 at 5 20 37 PM

final
Screenshot 2025-12-12 at 6 24 33 PM

@PickleNik PickleNik changed the title remove unused toc.tsx from landing, cleanup installation section tiny landing ui cleanup Dec 12, 2025
… on wider screens, add a new background pattern for one of the usecases cards
Comment thread packages/mcp-cloudflare/src/client/components/usecases/index.tsx Outdated
…anding and adjust sidebars and header-divider accordingly, fix install-tabs causing content overflow and remove unused sticky styles
…install-tabs a11y, disabled sidebar from being tabbable as it's redundant because there's a header button to trigger chat and it breaks the content flow for screen readers
@PickleNik PickleNik changed the title tiny landing ui cleanup landing ui cleanup, a11y improvements to tab navigation Dec 13, 2025
Comment thread packages/mcp-cloudflare/src/client/components/hero/hero-block.tsx Outdated
@dcramer

dcramer commented Dec 13, 2025

Copy link
Copy Markdown
Member

Way better - only concern I have is not being able to scroll left if you scroll right on the tabs for the diff installation options

@PickleNik

Copy link
Copy Markdown
Member Author

Way better - only concern I have is not being able to scroll left if you scroll right on the tabs for the diff installation options

right, I'm on trackpad so I assumed it'd just work the same when you scroll with the mouse my bad 😅

adding left scroll trigger that shows up once you scroll to the right

Screen.Recording.2025-12-13.at.2.49.06.PM.mov

Comment thread packages/mcp-cloudflare/src/client/app.tsx
@dcramer

dcramer commented Dec 15, 2025

Copy link
Copy Markdown
Member

lgtm!

@dcramer
dcramer merged commit 898adfe into getsentry:main Dec 15, 2025
9 of 10 checks passed
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