-
Notifications
You must be signed in to change notification settings - Fork 1.9k
[MacOS] Fixed the picker items do not appear when tapping on the picker after navigating through PushModalAsync #28003
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
[MacOS] Fixed the picker items do not appear when tapping on the picker after navigating through PushModalAsync #28003
Conversation
|
Hey there @NanthiniMahalingam! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed. |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
| App.Tap("Button"); | ||
| App.WaitForElement("Picker"); | ||
| App.Click("Picker"); | ||
| VerifyScreenshot(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pending snapshots (Mac and Windows). Already available in the latest build.
Could you commit the images?
Hi @jsuarezruiz ,
- Pending snapshots have been committed.
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/rebase |
76b7ba6 to
b2d553f
Compare
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/rebase |
b2d553f to
8041656
Compare
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/rebase |
8041656 to
11fa592
Compare
|
/azp run MAUI-UITests-public |
|
Azure Pipelines successfully started running 1 pipeline(s). |
| uITextField.EditingDidEnd += editingDidEndHandler; | ||
|
|
||
| var platformWindow = MauiContext?.GetPlatformWindow(); | ||
| var currentViewController = GetCurrentViewController(platformWindow?.RootViewController); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If platformWindow is null, the entire operation is unnecessary, so you can exit earlier to avoid unnecessary processing.
if (platformWindow == null) return;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If platformWindow is null, the entire operation is unnecessary, so you can exit earlier to avoid unnecessary processing.
if (platformWindow == null) return;
Hi @jsuarezruiz ,
I've added a null check for platformWindow to prevent unnecessary processing. Please review it.
|
/azp run MAUI-UITests-public |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/rebase |
cce2cc3 to
acf0700
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
|
/azp run MAUI-UITests-public |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run MAUI-public |
|
Azure Pipelines successfully started running 1 pipeline(s). |
…navigation (#28003) Added UI the test case. Added the outputs for android and iOS platform Updated the fix and added the output images Updated the code changes. Updated the output image


Issue Detail
Root Cause:
Description of Changes:
Issues Fixed
Fixes #6101
Fixes #21470
Validated the behaviour in the following platforms
Output
6101_before.mov
6101_after.mov