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

Added iOS 14 DatePickerStyle Support #488

Merged
merged 1 commit into from
Aug 29, 2020

Conversation

noorulain17
Copy link
Collaborator

  • Fixed crash in iOS 14.0
  • Added iOS 14 DatePickerStyle Support

@noorulain17 noorulain17 changed the title - Added iOS 14 DatePickerStyle Support Added iOS 14 DatePickerStyle Support Jul 8, 2020
+ (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
Copy link
Collaborator

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.

Copy link
Collaborator Author

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?

Copy link
Collaborator

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.

@ondererguven
Copy link

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 👼

@KunzManuel
Copy link
Collaborator

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 👍

@noorulain17
Copy link
Collaborator Author

@KunzManuel so can we merge this PR?

@KunzManuel
Copy link
Collaborator

@KunzManuel so can we merge this PR?
Yes

@noorulain17
Copy link
Collaborator Author

@KunzManuel can you please merge this PR? I don't have write access on this repository so can't merge.

@KunzManuel
Copy link
Collaborator

@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 ?

@ondererguven
Copy link

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?

@skywinder
Copy link
Owner

skywinder commented Aug 28, 2020

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 Would be much appreciated for help with this repo! Would you? @ondererguven @KunzManuel

@skywinder
Copy link
Owner

And thank you for keep updated!

- Added iOS 14 DatePickerStyle Support
@skywinder
Copy link
Owner

thanks, @noorulain17
Please, check email, I add you as a collaborator as we agreed in #348

Thank you for your support! 🎊

@skywinder skywinder self-requested a review August 29, 2020 11:14
Pickers/ActionSheetDatePicker.h Show resolved Hide resolved
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.

4 participants