fix(Disks): fix vdisk popup hiding #2946
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #2930
Stand (you need to turn on columns VDisks with PDisks): https://nda.ya.ru/t/kyq51ofu7Kj7fX
The problem was in different timeouts for
delayOpen
(100) anddelayClose
(200) in PDisk component, so the order of functions calls was wrongonShowPopup={() => setHighlightedVDisk(vDiskId)}
onHidePopup={() => setHighlightedVDisk(undefined)}
So highlighted VDisk ID always was
undefined
after mouse entered another PDisk.Introduced single constant for all disks popup in storage to fix this issue.
CI Results
Test Status:⚠️ FLAKY
📊 Full Report
Test Changes Summary ⏭️2
⏭️ Skipped Tests (2)
Bundle Size: ✅
Current: 85.59 MB | Main: 85.59 MB
Diff: +1.23 KB (0.00%)
✅ Bundle size unchanged.
ℹ️ CI Information