From 052d503dee6558adcc0aedf4438c581ce749fccf Mon Sep 17 00:00:00 2001 From: Doug Date: Thu, 7 Aug 2025 16:23:56 +0100 Subject: [PATCH] Fix the push animation when selecting a room with the compact layout on iPad. Also makes sure we don't animate the change of room in the split's detail when using the regular layout --- .../Sources/Application/Navigation/NavigationCoordinators.swift | 1 - ElementX/Sources/FlowCoordinators/RoomFlowCoordinator.swift | 1 - 2 files changed, 2 deletions(-) diff --git a/ElementX/Sources/Application/Navigation/NavigationCoordinators.swift b/ElementX/Sources/Application/Navigation/NavigationCoordinators.swift index 00386ce0e..b399d1c1e 100644 --- a/ElementX/Sources/Application/Navigation/NavigationCoordinators.swift +++ b/ElementX/Sources/Application/Navigation/NavigationCoordinators.swift @@ -760,6 +760,5 @@ private struct NavigationStackCoordinatorView: View { module.coordinator?.toPresentable() .id(module.id) } - .animation(.elementDefault, value: navigationStackCoordinator.rootModule) } } diff --git a/ElementX/Sources/FlowCoordinators/RoomFlowCoordinator.swift b/ElementX/Sources/FlowCoordinators/RoomFlowCoordinator.swift index 0adfd9329..0b94a8ef7 100644 --- a/ElementX/Sources/FlowCoordinators/RoomFlowCoordinator.swift +++ b/ElementX/Sources/FlowCoordinators/RoomFlowCoordinator.swift @@ -525,7 +525,6 @@ class RoomFlowCoordinator: FlowCoordinatorProtocol { roomScreenCoordinator = coordinator if !isChildFlow { - let animated = UIDevice.current.userInterfaceIdiom == .phone ? animated : false navigationStackCoordinator.setRootCoordinator(coordinator, animated: animated) { [weak self] in self?.stateMachine.tryEvent(.dismissFlow) }