-
Notifications
You must be signed in to change notification settings - Fork 38
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
Fixed navigationViewStyle and wrapped controller presentationStyle for iPad #2
Conversation
This shows the content as a single top view on iPad instead of a split view and simplifies navigation.
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.
Looks good to me!
As mentioned in #1, the later commits here fix the iPad presentation issue by using an explicit |
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.
Verified this works as expected on iPad and iPhone UIs.
* added link to branch * added link to branch * Update README.md * Fixed navigationViewStyle and wrapped controller presentationStyle for iPad (#2) * Use explicit navigationViewStyle on ContentView NavigationView. This shows the content as a single top view on iPad instead of a split view and simplifies navigation. * Use explicit modalPresentationStyle to fix iPad presentation issue. * Updated comment brought over from another sample. * Support for re-using existing shares on swift-concurrency branch. * Reuse existing CKShare if it already exists (#6) * Reuse existing CKShare if it already exists If a contact is already shared we use the existing CKShare object. fixes #5 * Rename func createShare * Rename ViewModelError `unknown` case Co-authored-by: Tony Tam <tonytam@apple.com> Co-authored-by: Tony Tam <tony@eatbacon.org> Co-authored-by: César Pinto Castillo <jagcesar@me.com>
By default iPad is using a split view so the contact list is only showing on the left in a split view. This sets an explicit
navigationViewStyle
to show the content as a single top view on iPad instead of a split view.Related to #1