Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reset preferences window animation #537

Closed
wants to merge 2 commits into from
Closed

reset preferences window animation #537

wants to merge 2 commits into from

Conversation

m5o
Copy link
Contributor

@m5o m5o commented May 21, 2019

🔗 #528

@m5o
Copy link
Contributor Author

m5o commented Jul 1, 2019

@jkbullard I know, is a dirty hack resetting the animation duration. What is your opinion on this?

@jkbullard
Copy link
Contributor

I agree, resetting the duration to zero is pretty hackish; one might just as well disable the code entirely. (Or even strip out the entire DBPrefsWindowController/MyPrefsWindowController classes, because as I recall the reason they were used was to do this animation.) So I'd prefer something that instead fixes the problem but preserves the automation; that is, that fades the old view out and then fades the new view in. (I'm assuming that the current code does both simultaneously and that's what's causing the problem.) That may be more trouble than it's worth but should be looked into.

I haven't seen this problem myself, although I've seen the screenshot in #522 and a movie of it happening, so it may depend on hardware speeds or something like that. (Although why Apple would make animation that depends on hardware speed I don't understand. Maybe it's due to a change in macOS's automation instead?)

@m5o
Copy link
Contributor Author

m5o commented Jul 3, 2019

Thank you for this detailed explanation. 🙇‍♂️

I do not have the Obj-C skills to refactor all the DBPrefsWindowController.m code, so I have to step back on this. Although remember the guidelines suggest changing different pref views without showing a "change in state".

@m5o m5o closed this Jul 3, 2019
@m5o m5o deleted the reset-preferences-window-viewanimation branch July 3, 2019 10:12
jkbullard added a commit that referenced this pull request Nov 1, 2019
…window

Similar to #537 by @m5o but changes only on Mojave and higher.

On macOS Mojave and higher, switching panels in the VPN Details window nowhappens instantly, which avoids the problem introduced in Mojave that caused macOS to ignore the "fade in" time.

On macOS El Capitan and lower, the panel switching behavior does not change: the panels are faded smoothly from one to another.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Text from two settings screens is displayed
2 participants