Repository navigation
ref(docs): Remove SplitLayout component and its remaining wrappers - #20017
Conversation
Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
The quick start template PRs (#19980-#19983) still have to remove the SplitLayout wrappers from 38 files. Until they merge, render those tags as plain pass-throughs so the build and preview succeed. Remove this shim once no MDX file uses the wrappers. Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
…ayout Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
This reverts commit 42a0c7b. Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
Removes the SplitLayout wrapper tags from the 38 files that the quick start template PRs also touch, and drops the temporary pass-through. No MDX file uses SplitLayout anymore, so this PR can merge first. Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
|
Pages that used a two-column split now show text and code as normal stacked docs. Monitoring in Services: Mention PlanWhat changedThe two-column split layout is removed from docs pages. RiskA missed wrapper tag makes the page fail to render for readers. Intended effectThe stacked layout itself has no metric.
Regression watchA failed render shows as a new React error or as a 404.
If the not-found rate rises, check logs for missing platform getting-started paths. Not observableThe visual switch from two columns to stacked content has no telemetry. |
Resolve conflict in python/tracing/streamed-spans/index.mdx by taking master's content (#19831) and removing its SplitLayout wrappers. Also remove SplitLayout wrappers from the new index__v2.x.mdx page added on master. Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
Resolve conflicts from #20017 (SplitLayout removal): keep this branch's content without the removed wrapper tags, and keep master's other changes to the same files. Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
Resolve conflicts from #20017 (SplitLayout removal): keep this branch's content without the removed wrapper tags, and keep master's other changes to the same files. Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
Resolve conflicts from #20017 (SplitLayout removal): keep this branch's content without the removed wrapper tags, and keep master's other changes to the same files. Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
Resolve conflicts from #20017 (SplitLayout removal): keep this branch's content without the removed wrapper tags, and keep master's other changes to the same files. Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
|
Verified in
|
DESCRIBE YOUR PR
Removes the
SplitLayoutcomponent everywhere. Pages that used it now render their text and code as regular docs content, with the standard docs spacing.<SplitLayout>,<SplitSection>,<SplitSectionText>, and<SplitSectionCode>wrapper lines from all 169 MDX files that used them, including the quick start guides. Only the tag lines and the blank lines around them change. Parsing each file before and after shows the same content once the wrappers are ignored.src/components/splitLayout/and its registration insrc/mdxComponents.ts..claude/skills/technical-docs/SKILL.md.Step numbering is unaffected.
StepConnectorfinds itsh2headings with a descendant selector, so it doesn't depend on these wrappers.This replaces #19984, which made SplitLayout render as a single column.
Merge order
Merge this first. After it merges, the quick start template PRs that also edit these files (#19980 Ruby, #19981 Dart and Flutter, #19982 Python, #19983 JavaScript) will need
mastermerged in. Those conflicts are limited to the removed wrapper tags.IS YOUR CHANGE URGENT?
PRE-MERGE CHECKLIST
via shannon.anahata.
--
View Junior Session [Sentry]