Skip to content

Replaced html snippets with a new ViewSourceCodeNotice component#4807

Merged
Duncanma merged 2 commits into
mainfrom
ViewSourceCodeNotice
Jul 1, 2026
Merged

Replaced html snippets with a new ViewSourceCodeNotice component#4807
Duncanma merged 2 commits into
mainfrom
ViewSourceCodeNotice

Conversation

@Duncanma

@Duncanma Duncanma commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

So I noticed the 'view this snippet in it's source repo' had some odd bottom margin spacing going on and make a change to the CSS to fix that... and then noticed it was a HTML snippet copied into a bunch (20-30) pages, so I turned it into a component ... and fixed the spacing issue. Otherwise there is no change to the behavior or appearance.

Before (live now)
image

After (preview)
image

┆Attachments: EDU-6636 Replaced HTML code snippets with a new ViewSourceCodeNotice component…

… in multiple documentation files across Go, Python, Java, and PHP SDKs to improve consistency and readability.
@vercel

vercel Bot commented Jul 1, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
temporal-documentation Ready Ready Preview, Comment Jul 1, 2026 10:47pm

Request Review

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

📖 Docs PR preview links

@Duncanma Duncanma marked this pull request as ready for review July 1, 2026 22:46
@Duncanma Duncanma requested a review from a team as a code owner July 1, 2026 22:46
@Duncanma Duncanma changed the title Replaced HTML code snippets with a new ViewSourceCodeNotice component… Replaced html snippets with a new ViewSourceCodeNotice component Jul 1, 2026
@Duncanma Duncanma enabled auto-merge (squash) July 1, 2026 22:51
@Duncanma Duncanma merged commit 61cc253 into main Jul 1, 2026
12 checks passed
@Duncanma Duncanma deleted the ViewSourceCodeNotice branch July 1, 2026 23:17
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