Add footprintAnimationDurationMultiplier default #1008
Conversation
|
Nice! I like it. |
|
I might add in a checkbox for this in Snap Areas tab, where checking the box would set it to something like 0.75 & unchecking would set it to 0. |
|
I agree. I understand you want to keep the settings minimalist but this one might be worth it. |
|
Might be cleaner to scale the multiplier so that the default when enabling it can be 1, and that gets scaled to whatever you decide the default speed should be. |
| box?.makeKeyAndOrderFront(nil) | ||
| box?.setFrame(newBoxRect, display: true) | ||
| if Defaults.footprintAnimationDurationMultiplier.value > 0 { | ||
| if !box!.isVisible, let origin = getFootprintAnimationOrigin(snapArea, newBoxRect) { |
There was a problem hiding this comment.
The animation's getting caught up here if the FootprintWindow hasn't closed yet (which only happens 0.3 seconds after the box.close() call), as it's still isVisible, even though it's completely transparent. So if you move a window away from a snapping area and straight back onto it within 0.3 seconds, the animation starts from where the window was when the window was moved away.
There was a problem hiding this comment.
Good catch. I have created a new PR.
Add optional animations to the footprint window.