Fix initiative effects renaming losing focus when set to 'Active and hover' #1773
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.
Initiative effects collapse when the mouse is not hovering over that entry. When editing an effect name for that entry, the effects would collapse when the mouse exited the parent entry even if the input box had focus.
Initiatially I tried to do this without hard coding separate 'mouseOver' and 'focused' statuses, something like
showEffect: { index: number, showCount: number }where the showCount would increment once for each event that asked for focus and decrement once when something released focus. But doing this, I ended up finding some edge cases where the effects would become stuck expanded and other entries could not become expanded. Due to this, I fell back on tracking the variables separately.This fixes #1767