javascript/NativeScript
1
0
Fork 0
mirror of https://github.com/NativeScript/NativeScript.git synced 2025-08-15 11:01:21 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(ios): prevent transitionCoordinator usage during modal presentation (#10153)

Browse Source 
closes https://github.com/NativeScript/NativeScript/issues/10115
This commit is contained in:
Nathan Walker
2023-01-03 15:01:21 -08:00
committed by GitHub
parent 760bbd06fa
commit d138ac000d
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
packages/core/ui/frame/index.ios.ts
View File

@ -118,7 +118,9 @@ export class Frame extends FrameBase {
if (!this._currentEntry) {
// Update action-bar with disabled animations before the initial navigation.
this._updateActionBar(backstackEntry.resolvedPage, true);
this.pushViewControllerAnimated(viewController, animated);
// Core defaults modalPresentationStyle to 1 for standard frame navigation
// for all others, it's modal presentation
this.pushViewControllerAnimated(viewController, animated, this._ios?.controller?.modalPresentationStyle !== 1);
if (Trace.isEnabled()) {
Trace.write(`${this}.pushViewControllerAnimated(${viewController}, ${animated}); depth = ${navDepth}`, Trace.categories.Navigation);
}
@ -180,13 +182,14 @@ export class Frame extends FrameBase {
}
}
private pushViewControllerAnimated(viewController: UIViewController, animated: boolean) {
private pushViewControllerAnimated(viewController: UIViewController, animated: boolean, isModal: boolean) {
let transitionCoordinator = this._ios.controller.transitionCoordinator;
if (transitionCoordinator) {
if (!isModal && transitionCoordinator) {
transitionCoordinator.animateAlongsideTransitionCompletion(null, () => {
this._ios.controller.pushViewControllerAnimated(viewController, animated);
});
} else {
// modal should always push immediately without transition coordinator
this._ios.controller.pushViewControllerAnimated(viewController, animated);
}
}