-
-
Notifications
You must be signed in to change notification settings - Fork 740
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
Added iOS 14 DatePickerStyle Support #488
Conversation
noorulain17
commented
Jul 8, 2020
- Fixed crash in iOS 14.0
- Added iOS 14 DatePickerStyle Support
+ (instancetype)showPickerWithTitle:(NSString *)title datePickerMode:(UIDatePickerMode)datePickerMode selectedDate:(NSDate *)selectedDate target:(id)target action:(SEL)action origin:(id)origin; | ||
|
||
+ (instancetype)showPickerWithTitle:(NSString *)title datePickerMode:(UIDatePickerMode)datePickerMode selectedDate:(NSDate *)selectedDate target:(id)target action:(SEL)action origin:(id)origin cancelAction:(SEL)cancelAction; | ||
+ (instancetype)showPickerWithTitle:(NSString *)title |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be nice to pass the datePickerStyle. Default could be .automatic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you prefer converting this class to Swift or do the datePickerStyle
changes in the current Objective C class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think Objective C should be enough in this case.
764a79a
to
c89e57c
Compare
Hey there, thanks for the work done here! Is there anyway I can help with this PR? We would like to update our project before iOS 14 launch 👼 |
From my point of view, it's done 👍 |
@KunzManuel so can we merge this PR? |
|
@KunzManuel can you please merge this PR? I don't have write access on this repository so can't merge. |
Neither do I, @ondererguven can merge this I guess ? |
I can't either because I don't have write access as well. @skywinder could you please merge this PR when you have the time? |
Hello, guys. I'm out of my mac for the next week. So can't even test it. If you are willing to help with maintaining this repo, I can add you as collaborators. |
And thank you for keep updated! |
- Added iOS 14 DatePickerStyle Support
c89e57c
to
3debe8f
Compare
thanks, @noorulain17 Thank you for your support! 🎊 |