-
-
Notifications
You must be signed in to change notification settings - Fork 480
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
添加 iPad 支持 #246
添加 iPad 支持 #246
Conversation
非常感谢!!我会找时间测试一下(需要点时间,最近比较忙),没有问题后会合并,再次感谢! |
我大致review了一下代码,基本没啥大问题,非常感谢你的pr! 已知的问题: 修复了操作提示不显示问题,将BarkSnackbarController 挪到了 UISplitViewController 的上一级。 还有就是我发现 UISplitViewController 并没有保存上一次打开的页面,但希望能和 StateStorageTabBarController 一样,保存上一次打开的页面,并且希望他们之间是能联动的,一个改变另外一个同时改变。 还有就是优化了一下切换页面频繁创建实例的问题,这一块我没仔细看,之后我可能会继续改一下,希望页面都能够复用。 |
好的呢,没想到这么快就有回应
要是能 Merge 进入 main 还是会感到很荣幸的 |
有道理,我已经将 UISplitViewController 改回iOS14的新版本了 同时初步添加了一下默认打开页面,不过还需要同步 BarkSplitViewController 中的 sectionViewController 和 snackBarController 的页面,现在切换时打开的页面不一致是各自独立的。 |
测试了一下好像没啥问题了,已经合并了。非常感谢!!。 |
🎉 |
看到 这个 issue有感,这么实用的应用如果能支持 iPad 就更好了,Swift/RX 苦手,略尽绵力。
代码改动似乎不小,以下希望能帮助 Review:
rx.modelSelected
改为rx.itemSelected
,改由增加传递 indexPath,方便 actionSheet popover 能正确定位 sourceView,否则 iPad 下单纯展示 actionSheet 会直接出现崩溃已自测已知路径,能正常支持 iPad 分屏模式:


