Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions
Description of the Change
Continuation of #380, with @Ben3eeE's comments addressed and the code updated.
Alternate Designs
I considered adding a global variable mouseActive to track if the mouse was in the tab bar or not, but it looks like mouseleave is buggy and does not fire a) when dragging or b) when right-clicking.
Benefits
"Smoother" animations.
Possible Drawbacks
This same exact code snippet now appears three times in the codebase.
Ben3eeE
commented
7 years ago
Requirements
Description of the Change
Continuation of #380, with @Ben3eeE's comments addressed and the code updated.
Alternate Designs
I considered adding a global variable
mouseActiveto track if the mouse was in the tab bar or not, but it looks likemouseleaveis buggy and does not fire a) when dragging or b) when right-clicking.Benefits
"Smoother" animations.
Possible Drawbacks
This same exact code snippet now appears three times in the codebase.
Applicable Issues
Fixes #371 Supersedes and closes #380
@Ben3eeE for testing