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

ObjectExplorer: Add a new Shortcut for UIWindow to change animations speed #629

Merged
merged 6 commits into from
Sep 26, 2022

Conversation

AnthoPakPak
Copy link
Contributor

This PR adds a new Shortcut to UIWindow to change its animations speed. It basically replicates the "Slow Animations" feature, only available on the Simulator. The user can set the speed of his choice.

As explained in the comments, as of now I've added it to UIView Shortcuts class instead of creating a new one for UIWindow. This is done like this to prevent UIWindow Shortcuts to override UIView ones (since as of now, an object can only get the Shortcuts from one class). Having a separate UIWindow class would have imply duplicating current UIView Shortcuts, which is not desirable. Tell me if you'd prefer any other way of handling this situation.

@NSExceptional
Copy link
Collaborator

Can we make this just a shortcut on UIWindow for the time being without modifying the view shortcuts file? One of them doesn't even work for UIWindow so we don't lose that much, right?

@AnthoPakPak
Copy link
Contributor Author

You're right, I did not noticed "Nearest VC" wasn't working on UIWindows.
Just made the changes to move this shortcut where it truly belongs.

@AnthoPakPak
Copy link
Contributor Author

Ugh, just clicked the wrong button while sending my comment, which closed the PR. Reopening.

@AnthoPakPak AnthoPakPak reopened this Sep 26, 2022
@NSExceptional NSExceptional merged commit c96df6d into FLEXTool:master Sep 26, 2022
@NSExceptional
Copy link
Collaborator

Thanks!

timonus pushed a commit to timonus/FLEX that referenced this pull request May 1, 2023
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.

2 participants