HomePhorge

Fixing a race condition for hovercards

Description

Fixing a race condition for hovercards

Summary:
Refs T1048 - Take this setup:

D123 D321

Hover with the mouse over one object, and move to another one before it got loaded. You'll have two hovercards. Of these you can't get rid of one anymore. Not through resizing, not through showing another hovercard. (You may have to try a few times). I think it only happens when #2 loads faster than #1. #2 is displayed, then #1 gets loaded and draws.

Or something like that. You could have a race condition where one draws after another and overwrites the current node/anchor, without hiding it. This renders the previous card unhideable, even on resize / mouse far away.

Test Plan:
Did a repro by hovering a lot.

Applied this change. Could not repro anymore.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Maniphest Tasks: T1048

Differential Revision: https://secure.phabricator.com/D5594

Details

Provenance
Anh Nhan NguyenAuthored on Apr 5 2013, 4:25 PM
epriestleyCommitted on Apr 5 2013, 4:25 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP886f3d507eff: Fix typo in PHP string
Branches
Unknown
Tags
Unknown

Event Timeline