-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Layout updates; widening content div, centering and responsiveness, scrollbar update #14766
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
Layout updates; widening content div, centering and responsiveness, scrollbar update #14766
Conversation
* 'master' of https://github.com/getsentry/sentry-docs: docs(dev-infra): Update development environment docs (#14585) add Sevvy integration docs (#14601) feat(dart/flutter): improve distinction between Logging integration and Structured Logs (#14527) correcting region storage for symbol and source map metadata (#14600)
…, Media Upload, Search Autocomplete) using existing Challenge/Solution/Frontend/Backend/How-it-works structure. Emphasizes where to instrument in React vs backend and low-cardinality attributes for span metrics.
…e auto-instrumentation; keep attributes minimal and business-focused for easier customization.
* 'master' of https://github.com/getsentry/sentry-docs: (22 commits) Fix incorrectly placed comma (#14592) fix(devservices): Typo in devservices docs (#14636) ref(ourlogs): Add log size limit (#14635) Document undocumented php prefix option (#14634) getsentry/relay@7958273 Unreal Engine: Update plugin docs (#14624) feat: Update min js sdk version for logging and update console logging snippet (#14614) feat: [Unreal] Add transaction and span set_data documentation (#14599) feat(js): Add `ignoreSpans` documentation to Options page (#14572) fix(js/browser): Amend user IP + `sendDefaultPii` change to v10 migration guide (#14609) fix: Fix broken AI banner due to invalid syntax (#14627) feat(partners)add-enablement-docs (#14095) Fix image path issue & add test coverage for new lightbox (#14616) feat(seer): update GHE setup to work with seer (#14617) fix(replay): Document searchable properties `screen` and alias `screens` (#14603) Update image handling (#14564) Update using-hooks.mdx (#14605) Godot: Document `disabled-in-editor-play` option (#14438) fix(Docs) redirect and deprecate old metrics pages (#14602) fix(relay): Update projects.mdx (#14606) ...
* 'master' of https://github.com/getsentry/sentry-docs: Add documentation for onDiscard callback in Java, Android, and Spring Boot (#14632) fix: Adding redirect for moved hydrogen docs to cloudflare (#14645)
* 'master' of https://github.com/getsentry/sentry-docs: (72 commits) docs(replay): Bring replay nav item to all apple platforms (#14710) User Feedback Basics Page (#14670) Update AM3 and legacy billing for logs GA, DO NOT MERGE BEFORE LOGS GA (#14589) Add LLMSteering component for hidden documentation instructions (#14744) New data retention policy page (#14661) Adding error prediction to Prevent AI docs (#14641) feat(logs): Remove beta note from logs docs (#14686) First pass at logs quota doc (#14671) docs(aws): Remove manual setup steps for ESM (#14738) Android Logs docs without Errors / Breadcrumbs distractions (#14733) chore(Unity): Dropped 2019 relevant section from Known Limitations (#14735) feat: Added Unity `UserFeedback` Prefab (#14667) chore(Unity): Added Mono & Line Numbers to Known Limitations (#14736) docs(js): Fix Next.js client tracing config (#14734) chore: Updated known limitation in Unity with Cysharp (#14666) feat(Unity ): Added `APK+OBB` initialization to troubleshooting (#14668) Codecov copy & screenshot edits (#14659) fix(cloudflare): Remove unused instrument.mjs from hono on cloudflare (#14723) update advanced scrubbing doc - adding logs section (#14696) fixing some errors from nextjs-source-map-updates branch (#14467) ...
* 'master' of https://github.com/getsentry/sentry-docs: feat(seer): Seer webhook docs (#14702) feat: Update logs spec to mention message template constraint (#14759) Unreal Engine: Update sampling docs (#14740) Unreal Engine: Update info about reading certain options from env vars (#14714) feat(Apple): Add `enableUnhandledCPPExceptionsV2` to experimental features (#14731) feat(kmp): Add Sentry Cocoa compatibility version info (#14737) Fix reference to span in processOrderBatch function (#14741) chore(java): Add docs on how to use tracePropagationTargets (#14752)
…C at large screens\n\n- Center content column by default and cap width via CSS var\n- At >=2057px, anchor left sidenav and right TOC to content edges\n- Add 5rem side padding on large screens; keep top padding unchanged\n- No changes to step or separator components in this commit
…crollbar with rounded thumb\n- Subtle hover/active contrast; transparent track\n- Firefox: use scrollbar-width: thin; themed colors\n
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Bundle ReportChanges will decrease total bundle size by 271.22kB (-1.19%) ⬇️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-client-array-pushAssets Changed:
Files in
view changes for bundle: sentry-docs-server-cjsAssets Changed:
Files in
App Routes Affected:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a much needed improvement! A couple of things though:
-
I'm noticing a very light gray background on the sidebar on light mode that appears like it's cut off on its left side. The right side has a border to separate it from the content, but the left side is pretty apparent so we can't leave it like that. Here's a zoomed in screenshot:

-
The TOC breaks between
1546pxand1296pxwidth. The main content and left sidebar remain the same width, so the TOC gets squeezed. Here's how it looks like on1400pxwidth:

- Add toc screen (1400px) in tailwind.config.mjs - Use hidden + toc:block on TOC aside - Ensures TOC stays hidden below 1400px and visible above
…tracing/span-metrics/examples.mdx from PR
… mode background via variable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good!
…yout width\n\n- Set to 1200px\n- Header max-width = sidebar + gap + content + gap + toc\n- Compensate asymmetry with translateX((toc - sidebar)/2)\n- Lock logo slot to sidebar width\n- Remove stray 24px padding and restore ~px-3 gutters across breakpoints
* 'master' of https://github.com/getsentry/sentry-docs: Layout updates; widening content div, centering and responsiveness, scrollbar update (#14766) Document default sdk logging attributes (#14353) Bump API schema to b8165143 (#14764) fix(OnboardingOptionButtons) Remove sticky css on mobile (#14690) Unreal Engine: Add migration guide (#14186) docs(ai): Update AI agent integrations list (#14769) fix: Order versioned docs select items by recency (#14768) ref(python): Update wording regarding async init (#14631) docs(aws-integration): Remove alert that ESM is not supported (#14756) fix(feedback): Update docs with SwiftUI notes (#14227) chore(docs): Remove explicit openai agents integration registration (#14718) Update list of states subject to tax on /pricing (#14763) ref(replay): Refactor pages inside the Product Walkthrough > Replay area (#14712) docs(replay): Move FAQ Replay for Android questions into dedicated troubleshooting page (#14709)


TDLR
Our content div was VERY narrow. Many of us have talked about how it feels cramped width wise. Widening that up to give a bit more space. Cleaning up the zone of emptiness that exists on the right in favor of centering the docs content on smaller resolutions, and centering all content on wider resolutions to improve UX.
Also centered header to match content width.
Summary
Motivation
Key Changes
Content
horizontal padding for sidenav and TOC
Navigation
right edge with a 24px gap.
Sidenav scrollbar
Header
Behavior by Breakpoint
Before -
Before widescreen -
After -
After widescreen -