Skip to content
This repository was archived by the owner on Nov 3, 2025. It is now read-only.

Conversation

@oscarcv
Copy link

@oscarcv oscarcv commented Nov 3, 2023

Summary

Fix for ios 17.1+ versions when displaying certain UIViewControllers where containerView has not yet been set.

Requirements (place an x in each [ ])

  • I've read and understood the Contributing Guidelines and have done my best effort to follow them.

  • I've read and agree to the Code of Conduct.

  • I've written tests to cover the new code and functionality included in this PR.

@oscarcv oscarcv mentioned this pull request Nov 3, 2023
9 tasks
@Pato-Salazar
Copy link

Pato-Salazar commented Nov 3, 2023

@TosinAF @ste57 Could any of you take a look at this small change to support iOS 17.1 and above? The fix is for this issue #202

@ste57
Copy link
Contributor

ste57 commented Nov 3, 2023

I don't think I have access to approve pull requests on this repo anymore - I may create a personal fork if this is causing frequent issues

@Pato-Salazar
Copy link

That could be cool. I think this is causing frequent issues.

@kwontaewan
Copy link

kwontaewan commented Nov 6, 2023

@oscarcv

The code doesn't work on iPad, do you know the reason?

@oscarcv
Copy link
Author

oscarcv commented Nov 6, 2023

@oscarcv

The code doesn't work on iPad, do you know the reason?

I have only tested it on iPhone. If you can make a basic project where the problem is reproduced I try to fix

@springfk
Copy link

does anybody find solution for iPad?

nil-bae added a commit to kidsnote/iOS-PanModal that referenced this pull request Dec 15, 2023
부터 dest viewController가 노출되지 않는 이슈 수정 사항 적용
(slackhq#204)
rahulVermaSimpplr added a commit to rahulVermaSimpplr/PanModal that referenced this pull request Sep 23, 2025
- picked code from PR
 - Fix ios 17.1+ issue slackhq#204
 - Update PanModalPresentationDelegate.swift slackhq#209
abdulazizSi added a commit to abdulazizSi/PanModal that referenced this pull request Sep 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants