-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
SwiftUI View #17
base: master
Are you sure you want to change the base?
SwiftUI View #17
Conversation
Next goal is to allow the user to select an item from the list
I think the next step is to get it working on iOS as well!
Hey @Meorge, thanks for the PR. I'll have a look at it in more detail over the next little bit and let you know what I've found. One thing I did notice though - the CI build failed as you're using |
Ah, thanks for noting that! I've never used a CI build system before, so I'm still learning a lot. I'll try upping the iOS version to 13.0. |
iOS 13+, as well as SwiftUI framework
This is strange - I tried updating the podspec file and pushing, but it seems to be stuck on an error on their end:
I'm guessing this just means I need to choose a different iOS version to target - one that they have the simulator for - but I'm not sure how to find out which ones are available. |
Ahhh @Meorge this might need some update on the stack used for the build. I think it might be using XCode 10. I'll look at it soon. |
Ah okay, thanks! |
Re-running it now with XCode 11.6 |
Apologies on being absent for a few days, there were some other things I had to work on. |
No worries at all, it's same with me - and also looks like I won't be able to get to this for another little while sorry. When I do get some time to review, I'll fix the Cocoapod issue as well. Sorry about the delay and thanks for contributing! |
Ok, no problem. Thanks again for putting up with my finding my way around the CI system 😅 |
Sorry both, but is the current version working with SwiftUI and iOS? |
Hi @jalurgemilang, sorry I've been busy and not really updating this library at all lately - at this stage I'm not sure I ever will really unless something changes. Feel free to fork it from the branch @Meorge has and use it from there rather than what's released. |
Hi @jalurgemilang , from my end I don't currently have plans to continue updating or working on it unless/until I have a project that needs it. I recall my implementation being fairly basic SwiftUI, so if it doesn't still work, it might not be a lot of effort to update it? |
Thanks, both of you for your comments! |
Here is the pull request for my SwiftUI version of the TimeZonePicker!
So far, it's worked correctly on macOS. I wasn't able to test it with iOS, unfortunately; for some reason Xcode refused to compile an iOS test app, and it looks like the solution may be to try and reinstall Xcode, which I'm not able to do at the moment. Looking through the code, though, I don't see anything that looks explicitly macOS-specific.
I also updated the README to include some info on using the SwiftUI view.