ref(issues): Remove streamlined UI from stack trace files#112242
ref(issues): Remove streamlined UI from stack trace files#112242
Conversation
github-actions
bot
added
the
Scope: Frontend
| setIsCollapsed(false); | ||
| } | ||
| // Expand the section | ||
| setIsCollapsed(false); |
There was a problem hiding this comment.
should this just be the new default
There was a problem hiding this comment.
I think this is correct as-is? it seems to be opening a different section on the page
| event={event} | ||
| meta={meta} | ||
| hideIcon={hasStreamlinedUI} | ||
| hideIcon |
There was a problem hiding this comment.
i think we can remove the icon then
There was a problem hiding this comment.
ah yes true, that allowed me to remove a lot more code related to the platform icon
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit f7af708. Configure here.
| <StackTraceContentPanel | ||
| className={wrapperClassName} | ||
| data-test-id="stack-trace-content" | ||
| hideIcon={hideIcon} |
There was a problem hiding this comment.
Wrapper components now serve no purpose after icon removal
Low Severity
The Wrapper styled component in both content.tsx and nativeContent.tsx existed solely to provide a position: relative containing block for the absolutely-positioned StacktracePlatformIcon. Now that the icon is removed, Wrapper is an unnecessary DOM node — Panel (which ContentPanel/StackTraceContentPanel extends) already has position: relative. Since this is a cleanup PR, these leftover wrappers are a natural candidate for removal.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit f7af708. Configure here.
2 participants
Continuing the code deletion of old issues UI