fix: resolve inconsistent tooltip display after drag and rapid hover#3907
fix: resolve inconsistent tooltip display after drag and rapid hover#3907akshay-gupta7 wants to merge 1 commit into
Conversation
Address three interacting issues in the tooltip stack:
1. DraggableWrapper: Synthesize pointerenter event after dragend to restore
hover state, since the browser doesn't automatically re-fire it after drag.
2. TokenTooltip: Keep tooltip mounted (pass empty div instead of empty string)
when edit form is open to prevent remount issues where pointerenter doesn't
fire on already-hovered elements.
3. Tooltip.Provider: Set skipDelayDuration={0} to prevent the default 300ms
delay from overwhelming rapid hover transitions between tokens.
Fixes: drag interactions breaking tooltips, rapid hover inconsistency, edit
form closing not restoring tooltips.
|
|
|
|
Commit SHA:3ec9c55d259152923eedb3f79b9071ac891eb103 Test coverage results 🧪
|
|
Commit SHA:3ec9c55d259152923eedb3f79b9071ac891eb103 |
|
🛡️ Hyma Compliance Check ✅ No compliance-relevant changes detected 8 file(s) scanned · 0 findings All changes are UI rendering logic (tooltip display, drag event handling, synthetic pointer events, whitespace cleanup, test formatting). No new data processing, no external calls, no access control changes, no new packages, no credentials.
|
|
🛡️ Hyma Compliance Check ✅ No compliance-relevant changes detected Files changed in this PR were reviewed against the compliance taxonomy (ROPA, Subprocessor, Access Control, AI Provider Routing, Secrets/Credentials, Policy Gap). No findings were identified.
|
|
🛡️ Hyma Compliance Check ✅ No compliance-relevant changes detected 9 file(s) scanned · 0 findings
|
|
🛡️ Hyma Compliance Check ✅ No compliance-relevant changes detected Files scanned · 0 findings No new external calls, personal data processing changes, or authentication/authorization modifications detected. No compliance action required.
|
|
🛡️ Hyma Compliance Check ✅ No compliance-relevant changes detected 9 file(s) scanned · 0 findings
|
|
🛡️ Hyma Compliance Check ✅ No compliance-relevant changes detected 4 file(s) scanned · 0 findings
|
2 participants
Address three interacting issues in the tooltip stack:
DraggableWrapper: Synthesize pointerenter event after dragend to restore hover state, since the browser doesn't automatically re-fire it after drag.
TokenTooltip: Keep tooltip mounted (pass empty div instead of empty string) when edit form is open to prevent remount issues where pointerenter doesn't fire on already-hovered elements.
Tooltip.Provider: Set skipDelayDuration={0} to prevent the default 300ms delay from overwhelming rapid hover transitions between tokens.
Fixes: drag interactions breaking tooltips, rapid hover inconsistency, edit form closing not restoring tooltips.
Why does this PR exist?
Closes #3891
What does this pull request do?
Testing this change
Additional Notes (if any)