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

Significantly improved panning performance #445

Closed
wants to merge 1 commit into from
Closed

Significantly improved panning performance #445

wants to merge 1 commit into from

Conversation

fluidsonic
Copy link
Contributor

… when not using centerViewCornerRadius at all by not applying a mask to the IIViewDeckView.

Applying a mask to IIViewDeckView is a really bad idea in any case because it significantly hurts drawing performance! For now I just fixed it for the case where no corner radius was set at all.

…ornerRadius at all by not applying a mask to the IIViewDeckView.

Applying a mask to IIViewDeckView is a really bad idea in any case because it significantly hurts drawing performance!
@itsthejb
Copy link

itsthejb commented Feb 4, 2014

This seems to disable the centre controller shadow altogether if a corner radius is not set. Don't think this is what you intended... It certainly improves performance, though; because there's no shadow!

@MadeByDouglas
Copy link

are there any plans to update this with UIKit Dynamics? like what this guy built https://github.com/erichoracek/MSDynamicsDrawerViewController ?

@michaelochs
Copy link
Member

@Audentia I am currently working on v3.0.0 which basically is a reimplementation of the API. I haven't thought about using UIKit Dynamics but I am happy to discuss the pros and cons about this. Feel free to open a feature request!

@MadeByDouglas
Copy link

@michaelochs hey awesome! Yeah I've only dabbled with UIKit Dynamics so I'm not sure all the pros cons myself, but my understanding is its Apple's preferred way for UI animations now. I saw that other guys project and it looks promising, but he hasn't updated it in a while so was hesitant to use it in our production app. These are all purely selfish motivations, we are using viewdeck in our app and im just seeing how to make it the best it can be :) If i get time to dive in and find specific benefits of Dynamics I will happily open a feature request, and if really dive in, I'll submit a pull request :)

@michaelochs
Copy link
Member

Closing this PR for now.

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.

None yet

5 participants