codemod(flex): apply changes for @getsentry/replay-frontend #106837
Conversation
github-actions
bot
added
the
Scope: Frontend
![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 3 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| `; | ||
| export function OverflowHidden(props: ContainerProps<'div'>) { | ||
| return <Container height="100%" overflow="hidden" position="relative" {...props} />; | ||
| } |
There was a problem hiding this comment.
Missing display grid in OverflowHidden component
High Severity
The OverflowHidden component is missing display="grid" in its conversion from a styled component. The original styled component had display: grid which is critical for the layout to work correctly with AutoSizer and MultiGrid components that depend on this grid layout.
| {...props} | ||
| /> | ||
| ); | ||
| } |
There was a problem hiding this comment.
Missing display grid in SplitPanel component
High Severity
The SplitPanel component is missing display="grid" in its conversion from a styled component. The original had display: grid which is essential because the component is used with gridTemplateRows style prop in network/index.tsx, and grid template properties only work when display is set to grid.
| <ErrorTable data-test-id="replay-details-errors-tab"> | ||
| {errorFrames ? ( | ||
| <OverflowHidden> | ||
| <Container height="100%" overflow="hidden" position="relative"> |
There was a problem hiding this comment.
Missing display grid in errorList Container
High Severity
The Container replacement for the removed OverflowHidden styled component is missing display="grid" property. The original styled component had display: grid which is required for the AutoSizer and MultiGrid layout to function correctly.
gggritso
left a comment
gggritso
left a comment
There was a problem hiding this comment.
At a glance those Bugbot complaints look legitimate, otherwise this LGTM!
Applies the flex codemod to files owned by @getsentry/replay-frontend.