feat(issues): Cleanup debug meta processing column#107384
Conversation
There was a bunch of exported components that are basically just `Flex`. Wraps the entire column in the tooltip instead of just the icon.
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 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| <Text size="sm">{t('Stack Unwinding')}</Text> | ||
| </Flex> | ||
| </Tooltip> | ||
| )} |
There was a problem hiding this comment.
Missing tooltip disabled prop for falsy title values
Low Severity
The Tooltip components are missing the disabled prop that was present in the old code (disabled={!details}). The getProcessingInfoTooltip function returns undefined for OK status and for ERROR/MALFORMED when details is undefined. Based on codebase patterns (e.g., button.tsx uses disabled={!title}), the Tooltip component needs explicit disabling when title is falsy. Without this, empty tooltips may appear on hover for items with OK status or when error details are not available.
thetruecpaul
left a comment
thetruecpaul
left a comment
There was a problem hiding this comment.
LGTM — nice cleanup!
There was a bunch of exported components that are basically just `Flex`. Wraps the entire column in the tooltip instead of just the icon. before <img width="867" height="278" alt="image" src="https://github.com/user-attachments/assets/1ffec1ec-5af5-4bb2-ad46-ac7e66dc5815" /> after <img width="671" height="222" alt="image" src="https://github.com/user-attachments/assets/d4746f4f-9034-4a6b-97b3-4a57ad8751d3" />
There was a bunch of exported components that are basically just `Flex`. Wraps the entire column in the tooltip instead of just the icon. before <img width="867" height="278" alt="image" src="https://github.com/user-attachments/assets/1ffec1ec-5af5-4bb2-ad46-ac7e66dc5815" /> after <img width="671" height="222" alt="image" src="https://github.com/user-attachments/assets/d4746f4f-9034-4a6b-97b3-4a57ad8751d3" />
2 participants
There was a bunch of exported components that are basically just
Flex. Wraps the entire column in the tooltip instead of just the icon.before
after