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

[WIP] For iOS13 do not use automatic modal presentation style for segue #617

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

phimage
Copy link
Member

@phimage phimage commented Aug 23, 2019

Segues set using storyboards, such as PresentSlideSegue, seem to failed to present

I see that iOS 13 add new navigation stack and automatic modal presentation style
Before fullscreen(=0) was the default one (If I debug)

I am wondering If IBAnimatable must force to fullscreen the modal presentation style like I do in my PR when the segue is used (to make it work)

or document and say you must select Presentation: Full Screen (or current context if previous was also full screen etc...)
Screenshot 2019-08-23 at 10 32 14
(xcode 11 screenshot, must not be available in xcode 10)

@IBAnimatableBot
Copy link

IBAnimatableBot commented Aug 23, 2019

1 Error
🚫 Any changes to library code need a summary in the Changelog.
3 Warnings
⚠️ PR is classed as Work in Progress
⚠️ Consider adding supporting documentation to this change. Documentation can be found in the docs directory.
⚠️ Consider adding / updating the demo app.

Generated by 🚫 Danger

@tbaranes
Copy link
Member

I do think you have the right approach, won't we have issue with all the other segue overriden? I don't remember exactly how it was working.

@JakeLin
Copy link
Member

JakeLin commented Aug 24, 2019

Because Segue doesn't support @IBInspectable , we have to set a default value for one Segue, the walkaround is to create different segue for different Presentation.

If the current Segues only support fullscreen, shall we change the method func present(animated flag: Bool = true) to func presentFullScreen(animated flag: Bool = true)?

@phimage
Copy link
Member Author

phimage commented Aug 24, 2019

I can rename but I will make my mode less intrusive, so it is not always fullscreen

Set fullscreen only if in automatic, so if someone choose explicitly an other presentation it will continue to work. And if he want to have issues he can choose pageSheet :)
But to compile with xcode 10.3 I must do something, maybe use rawValue of automatic of check on swift version 5.1...

@phimage phimage changed the title For iOS13 do not use automatic modal presentation style for segue [WIP] For iOS13 do not use automatic modal presentation style for segue Aug 24, 2019
@phimage
Copy link
Member Author

phimage commented Aug 29, 2019

I has made my code more restrictive to fix the segue but!

I see that fixing the segue is not the solution
In you test the demo app, which is not using segue, the Transition part failed to "present" things too. The same code is used after all

Maybe some code to fix into UIPresentationController override or delegates

@mingsai
Copy link

mingsai commented Jan 10, 2020

In Xcode 11.3, I had to apply the Presentation fullscreen setting onto specific View Controllers in order for the segue to complete successfully. Making sure to use the custom segues from IBAnimatable now works again.

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.

None yet

6 participants