Skip to content

docs(js): Use split layout in Angular & SvelteKit quick start guide#17227

Merged
inventarSarah merged 4 commits intomasterfrom
smi/quick-start/split-layout-ang-sveki
Apr 7, 2026
Merged

docs(js): Use split layout in Angular & SvelteKit quick start guide#17227
inventarSarah merged 4 commits intomasterfrom
smi/quick-start/split-layout-ang-sveki

Conversation

@inventarSarah
Copy link
Copy Markdown
Collaborator

DESCRIBE YOUR PR

This branch updates the following quick start guides to use the new split layout:

  • Angular wizard
  • Angular manual setup
  • SvelteKit wizard
  • SvelteKit manual setup
  • SvelteKit on Cloudflare

I created temporary include files for tunneling and source maps content (which I will clean up once all guides are updated). I then also updated the Browser JS quick start guide to use these new include files.

Closes: #17187

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.
    Thanks in advance for your help!

PRE-MERGE CHECKLIST

Make sure you've checked the following before merging your changes:

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

EXTRA RESOURCES

@inventarSarah inventarSarah requested review from JPeer264 and Lms24 April 2, 2026 07:32
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview, Comment Apr 2, 2026 7:32am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored Apr 2, 2026 7:32am

Request Review

Copy link
Copy Markdown
Contributor

@coolguyzone coolguyzone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for updating these!

Copy link
Copy Markdown
Member

@JPeer264 JPeer264 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uh that looks way better. Thanks for updating

Copy link
Copy Markdown
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Content-wise, this LGTM. I agree that it looks a bit better than before. However, I think we have some opportunities to clean up a lot of wasted space in the manual setup guides. For example, the CTAs to log in seem highly duplicated and push down the code blocks quite a bit:

Image

Doesn't have to block this PR as this likely is a problem everywhere. But long-term, we should come up with a smarter concept here how to display this.

@inventarSarah
Copy link
Copy Markdown
Collaborator Author

inventarSarah commented Apr 7, 2026

@Lms24

However, I think we have some opportunities to clean up a lot of wasted space in the manual setup guides. For example, the CTAs to log in seem highly duplicated and push down the code blocks quite a bit

Yes, these come from the Code block component and are everywhere.
I took a closer look, and it doesn't appear to be working as intended. For example, it shouldn't appear in the code block on your screenshot, since there's no config to replace in it. (This is also the case in master branch)
I'll create a new issue for this, thanks!

@inventarSarah inventarSarah merged commit ad03a25 into master Apr 7, 2026
18 checks passed
@inventarSarah inventarSarah deleted the smi/quick-start/split-layout-ang-sveki branch April 7, 2026 09:30
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.

Angular & SvelteKit quick start guide split layout

4 participants