Skip to content

fix(issues): Simplify stack trace code mapping modal, smaller button#119790

Draft
scttcper wants to merge 2 commits into
masterfrom
scttcper/code-mapping-setup
Draft

fix(issues): Simplify stack trace code mapping modal, smaller button#119790
scttcper wants to merge 2 commits into
masterfrom
scttcper/code-mapping-setup

Conversation

@scttcper

@scttcper scttcper commented Jul 15, 2026

Copy link
Copy Markdown
Member

makes the code mapping setup action collapse to a code icon when the frame header gets tight, and cleans up the modal so it points at the connected project and handles long file paths without weird wrapping.

the setup copy now describes suspect commits and source code links, with a dedicated copyable path surface.

before

image

after (smaller screen)

image

after (large screen)

image

before

Screenshot 2026-07-15 at 4 28 11 PM

after

Screenshot 2026-07-15 at 4 09 12 PM

The setup action was taking too much stack trace header space on smaller frames, and the modal repeated long file paths in copy that wrapped badly.

Use a compact code icon at narrow container sizes and make the setup flow clearer about the repository, file path, and URL it needs. Long paths now get a dedicated wrapping surface with a pinned copy action.

Co-Authored-By: OpenAI Codex <noreply@openai.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 15, 2026
Use a supported Text element while keeping the path monospace.

Co-Authored-By: OpenAI Codex <noreply@openai.com>
@github-actions

Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

Metric Before After Delta
Coverage 93.97% 93.97% ±0%
Typed 135,521 135,531 🟢 +10
Untyped 8,692 8,692 ±0
🔍 5 new type safety issues introduced

Non-null assertions (!) (5 new)

File Line Detail
static/app/components/events/interfaces/frame/stacktraceLink.tsx 232 sourceCodeProvider?.provider.key!
static/app/components/events/interfaces/frame/stacktraceLinkModal.tsx 117 sourceCodeIntegration!
static/app/components/events/interfaces/frame/stacktraceLinkModal.tsx 120 sourceCodeIntegration!
static/app/components/events/interfaces/frame/stacktraceLinkModal.tsx 269 sourceCodeIntegration!
static/app/components/events/interfaces/frame/stacktraceLinkModal.tsx 292 sourceCodeIntegration!

This is informational only and does not block the PR.

@scttcper scttcper changed the title fix(issues): Improve code mapping setup fix(issues): Simplify stack trace code mapping setup Jul 15, 2026
@scttcper scttcper changed the title fix(issues): Simplify stack trace code mapping setup fix(issues): Simplify stack trace code mapping modal, smaller button Jul 15, 2026
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.

1 participant