Orderella / PopupDialog

A simple, customizable popup dialog for iOS written in Swift. Replaces UIAlertController alert style.
http://www.mwfire.de
Other
3.95k stars 521 forks source link

Transitions don't use "inDuration" #357

Open BlueCobold opened 4 years ago

BlueCobold commented 4 years ago

224 Report

All transitions use hardcoded "inDuration" of 0.6s, although they have a constructor-parameter for it. It would further be great to have them flexible, because 0.6s is really slow for some situations and animations.

Environment

Dependency management

What did you expect to happen?

I expect to use animations to use the given constructor-parameters for animation durations or potentially be configurable

What happened instead?

They use 0.6s hardcoded in any case https://github.com/Orderella/PopupDialog/blob/014f7b2fe3e907c5c0cd20780eb91f458fc7655d/PopupDialog/Classes/TransitionAnimations.swift#L89 https://github.com/Orderella/PopupDialog/blob/014f7b2fe3e907c5c0cd20780eb91f458fc7655d/PopupDialog/Classes/TransitionAnimations.swift#L120 https://github.com/Orderella/PopupDialog/blob/014f7b2fe3e907c5c0cd20780eb91f458fc7655d/PopupDialog/Classes/TransitionAnimations.swift#L151

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.