Skip to content

ref(autofix): Use new Markdown primitive in v3 cards#115879

Merged
priscilawebdev merged 1 commit into
masterfrom
priscila/ref/autofix-use-new-markdown-for-suggested-fix
May 22, 2026
Merged

ref(autofix): Use new Markdown primitive in v3 cards#115879
priscilawebdev merged 1 commit into
masterfrom
priscila/ref/autofix-use-new-markdown-for-suggested-fix

Conversation

@priscilawebdev
Copy link
Copy Markdown
Member

Replace the local StyledMarkedText wrapper (built on the legacy sentry/utils/marked MarkedText component) with the new <Markdown /> primitive from @sentry/scraps/markdown introduced in #115025.

This is the autofix surface rendered in the issue details sidebar and the Seer drawer — the Solution card ("Plan"), Root Cause card, and the streaming loading details. The v3/styled.tsx wrapper is no longer needed and is removed.

No behavior change is intended. The new primitive renders each markdown block via Scraps Text/Heading/InlineCode/etc. instead of dangerouslySetInnerHTML, so visual rendering of the one-line summaries and lists in RootCauseCard / SolutionCard may shift slightly — please eyeball the cards in the sidebar and drawer during review.

Refs #115025

Replace the local StyledMarkedText wrapper (built on the legacy
sentry/utils/marked MarkedText component) with the new <Markdown />
primitive from @sentry/scraps/markdown introduced in #115025. Covers
the Solution and Root Cause cards rendered in the issue details
sidebar and Seer drawer, plus the streaming loading details. The
shared v3/styled.tsx wrapper is no longer needed and is removed.
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.56%

@priscilawebdev
Copy link
Copy Markdown
Member Author

I still need to understand why the Stack constrains the text to a fixed width instead of allowing it to use the full available horizontal space, but this PR already ships a few improvements, such as:

Before
image

After

image

@priscilawebdev priscilawebdev marked this pull request as ready for review May 21, 2026 09:22
@priscilawebdev priscilawebdev requested a review from a team as a code owner May 21, 2026 09:22
@priscilawebdev priscilawebdev requested review from a team and natemoo-re May 21, 2026 09:22
Copy link
Copy Markdown
Member

@natemoo-re natemoo-re left a comment

Choose a reason for hiding this comment

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

🙌 Thanks!

@priscilawebdev priscilawebdev merged commit 024f18b into master May 22, 2026
72 checks passed
@priscilawebdev priscilawebdev deleted the priscila/ref/autofix-use-new-markdown-for-suggested-fix branch May 22, 2026 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants