Skip to content

Conversation

@SuthiYuvaraj
Copy link
Contributor

@SuthiYuvaraj SuthiYuvaraj commented Apr 4, 2025

Reverts: #28534

Description for revert:

This PR has been reverted due to differing behavior on iOS when dynamically changing the page associated with a Window. In the HostApp, the page is directly assigned to the Window. When running a test case, the selected item from the test case is assigned to Application.Current.Windows[0].Page.

During this page swap, the Page Mapper is triggered before the old page is removed and the new page is added. As a result, the PropertyMapper executes early, and the HideSoftInputOnTapped property is updated. This property relies on the Page.Parent, which is null at that point, causing a failure on iOS.

Meanwhile, Android updates the page properties only after the previous page has been removed and the new page has been added to the window, which avoids this issue

@dotnet-policy-service
Copy link
Contributor

Hey there @@SuthiYuvaraj! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@dotnet-policy-service dotnet-policy-service bot added community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration labels Apr 4, 2025
@SuthiYuvaraj SuthiYuvaraj marked this pull request as ready for review April 7, 2025 04:47
@SuthiYuvaraj SuthiYuvaraj requested a review from a team as a code owner April 7, 2025 04:47
@SuthiYuvaraj SuthiYuvaraj requested review from PureWeen and mattleibow and removed request for a team April 7, 2025 04:47
@PureWeen PureWeen changed the base branch from inflight/current to inflight/candidate April 8, 2025 19:41
@PureWeen
Copy link
Member

PureWeen commented Apr 8, 2025

/rebase

@PureWeen PureWeen force-pushed the revert-28534-fix-26792-preview branch from 9b13427 to 77b05e0 Compare April 8, 2025 19:43
@PureWeen PureWeen merged commit c65c3fb into dotnet:inflight/candidate Apr 8, 2025
1 check passed
github-actions bot pushed a commit that referenced this pull request Apr 11, 2025
github-actions bot pushed a commit that referenced this pull request Apr 14, 2025
github-actions bot pushed a commit that referenced this pull request Apr 15, 2025
@github-actions github-actions bot locked and limited conversation to collaborators May 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration platform/android test-flight

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants