-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
fix(ui): Only copy CodeBlock expandable content #13973
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
fix(ui): Only copy CodeBlock expandable content #13973
Conversation
Prior to this change all `code` elements would be copied, so if the markdown content inside a expandable alert included single-backtick style `<code>whatever</code>` nodes, those would end up in the copied content. The intention of this component is definitely intended to just copy the CodeBlock content.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Bundle ReportChanges will increase total bundle size by 49 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-server-cjsAssets Changed:
Files in
App Routes Affected:
view changes for bundle: sentry-docs-client-array-pushAssets Changed:
Files in
|
|
Dupe of #13972 - we could do both changes, though I'm not sure this change with the way its struct ured wouldn't break anything else |
Pull request was closed
|
Cursor change seems fine for now |
Prior to this change all
codeelements would be copied, so if themarkdown content inside a expandable alert included single-backtick
style
<code>whatever</code>nodes, those would end up in the copiedcontent.
The intention of this component is definitely intended to just copy the
CodeBlock content.