Skip to content

Jira Issue selection tooltip fix#1231

Merged
arnestrickmann merged 1 commit intogeneralaction:mainfrom
prath47:fix-1005
Mar 2, 2026
Merged

Jira Issue selection tooltip fix#1231
arnestrickmann merged 1 commit intogeneralaction:mainfrom
prath47:fix-1005

Conversation

@prath47
Copy link
Contributor

@prath47 prath47 commented Mar 2, 2026

fix: #1005

did the same changes for jira as well

@vercel
Copy link

vercel bot commented Mar 2, 2026

@prath47 is attempting to deploy a commit to the General Action Team on Vercel.

A member of the Team first needs to authorize it.

@prath47 prath47 changed the title jira issue fix Jira Issue selection tooltip fix Mar 2, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 2, 2026

Greptile Summary

Applied the same tooltip singleton management pattern from LinearIssuePreviewTooltip to JiraIssuePreviewTooltip. This ensures only one tooltip can be open at a time across all Jira issue preview instances.

  • Added module-level singleton activeTooltipForceClose to track the currently open tooltip
  • Implemented controlled tooltip state with open and delayed close timing (300ms)
  • Added mouse enter/leave handlers on both trigger and content to prevent premature closing
  • Used React refs pattern for stable force-close callback across renders

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The implementation directly mirrors the proven pattern from LinearIssuePreviewTooltip. The changes are purely additive UI improvements with no impact on business logic, data, or external systems. The singleton pattern is correctly implemented with proper cleanup handlers.
  • No files require special attention

Important Files Changed

Filename Overview
src/renderer/components/JiraIssuePreviewTooltip.tsx Added singleton tooltip management pattern to prevent multiple tooltips from being open simultaneously, mirroring the LinearIssuePreviewTooltip implementation

Last reviewed commit: d21723c

@arnestrickmann
Copy link
Contributor

Thanks sir! @prath47

@arnestrickmann arnestrickmann merged commit 2f82e57 into generalaction:main Mar 2, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug]: Not able to hover over modal of extra info about github issues

2 participants