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

Dismissal animation glitch #3

Open
fl034 opened this issue Sep 17, 2019 · 8 comments
Open

Dismissal animation glitch #3

fl034 opened this issue Sep 17, 2019 · 8 comments

Comments

@fl034
Copy link

fl034 commented Sep 17, 2019

ezgif-3-69a238546023

I slowed down the animation to make this glitch better visible.
It happens at the handover of the view from the interactive and non-interactive part of the transition.

I assume the problem lies around here, but I can't figure it out

// Card fills inside animated container view
cardDetailView.edges(to: animatedContainerView)
animatedContainerView.centerXAnchor.constraint(equalTo: container.centerXAnchor).isActive = true
let animatedContainerTopConstraint = animatedContainerView.topAnchor.constraint(equalTo: container.topAnchor, constant: params.settings.cardContainerInsets.top)
let animatedContainerWidthConstraint = animatedContainerView.widthAnchor.constraint(equalToConstant: cardDetailView.frame.width - (params.settings.cardContainerInsets.left + params.settings.cardContainerInsets.right))
let animatedContainerHeightConstraint = animatedContainerView.heightAnchor.constraint(equalToConstant: cardDetailView.frame.height - (params.settings.cardContainerInsets.top + params.settings.cardContainerInsets.bottom))
NSLayoutConstraint.activate([animatedContainerTopConstraint, animatedContainerWidthConstraint, animatedContainerHeightConstraint])

@chelemen-razvan
Copy link
Contributor

chelemen-razvan commented Sep 18, 2019

I'll look into it, but right now fixing the ios 13 issues are more important. If you think you can fix it yourself feel free to make a pull request.

@fl034
Copy link
Author

fl034 commented Sep 26, 2019

Maybe I find time to look into it. It looks okay for now if the animation is now too slow ;)
In general your code works fine on iOS 13, btw.

@chelemen-razvan
Copy link
Contributor

Not if you compile it with XCode 11

@fl034
Copy link
Author

fl034 commented Sep 26, 2019

I'm was using Xcode 11 and 11.1 and it's working fine (except my two issues). What do you mean?

@chelemen-razvan
Copy link
Contributor

I don't know yet, but the insets behaviour seem to be different.

@Abhi-philips
Copy link

ezgif-3-69a238546023

I slowed down the animation to make this glitch better visible.
It happens at the handover of the view from the interactive and non-interactive part of the transition.

I assume the problem lies around here, but I can't figure it out

// Card fills inside animated container view
cardDetailView.edges(to: animatedContainerView)
animatedContainerView.centerXAnchor.constraint(equalTo: container.centerXAnchor).isActive = true
let animatedContainerTopConstraint = animatedContainerView.topAnchor.constraint(equalTo: container.topAnchor, constant: params.settings.cardContainerInsets.top)
let animatedContainerWidthConstraint = animatedContainerView.widthAnchor.constraint(equalToConstant: cardDetailView.frame.width - (params.settings.cardContainerInsets.left + params.settings.cardContainerInsets.right))
let animatedContainerHeightConstraint = animatedContainerView.heightAnchor.constraint(equalToConstant: cardDetailView.frame.height - (params.settings.cardContainerInsets.top + params.settings.cardContainerInsets.bottom))
NSLayoutConstraint.activate([animatedContainerTopConstraint, animatedContainerWidthConstraint, animatedContainerHeightConstraint])

did you solved dismissal animation glitch

@Abhi-philips
Copy link

I am also facing the same issue

@fl034
Copy link
Author

fl034 commented Mar 5, 2020

@Abhi-philips I couldn't fix it. I speeded up the animation so it didn't bother me anymore :D. But would be cool if this would be fixed, maybe by @chelemen-razvan

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

No branches or pull requests

3 participants