Skip to content

[Android] Picker - focus/unfocus events #28122

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

Conversation

kubaflo
Copy link
Contributor

@kubaflo kubaflo commented Mar 1, 2025

Issues Fixed

Fixes #28121
Fixes #21704
Fixes #19739

Before After
Screen.Recording.2025-03-01.at.20.22.09.mov
Screen.Recording.2025-03-01.at.20.21.02.mov

@kubaflo kubaflo requested a review from a team as a code owner March 1, 2025 19:25
@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Mar 1, 2025
@kubaflo kubaflo added platform/android area-controls-picker Picker community ✨ Community Contribution and removed community ✨ Community Contribution labels Mar 1, 2025
@kubaflo kubaflo force-pushed the Picker-Focused/Unfocused-Events-Do-Not-Fire-on-Android- branch from d53d4d5 to d1a6b33 Compare March 1, 2025 23:44
@kubaflo
Copy link
Contributor Author

kubaflo commented Mar 2, 2025

BTW bumping this time and date picker focus/unfocus events related PR of mine :)
#20547

@kubaflo kubaflo force-pushed the Picker-Focused/Unfocused-Events-Do-Not-Fire-on-Android- branch from d1a6b33 to abf3a49 Compare March 2, 2025 16:01
@jsuarezruiz
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@@ -0,0 +1,33 @@
#if ANDROID
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While the fix is for Android, I think the test could run on all the platforms and verify the same events everywhere. Could we remove the compilation directive?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2025-03-03 at 14 27 56

It won't work because even with such addition

#if IOS
    App.Click("Done");
#endif

the iOS picker won't close

@bhavanesh2001
Copy link
Contributor

@kubaflo I think this will fix #19739

pickers.mov

@kubaflo
Copy link
Contributor Author

kubaflo commented Mar 25, 2025

@kubaflo I think this will fix #19739

pickers.mov

Thanks for letting me know!

@kubaflo kubaflo force-pushed the Picker-Focused/Unfocused-Events-Do-Not-Fire-on-Android- branch from abf3a49 to e7ddc66 Compare March 30, 2025 23:40
@kubaflo kubaflo self-assigned this Mar 30, 2025
@karthikraja-arumugam
Copy link
Contributor

This also fixes the issue #8546

@rmarinho
Copy link
Member

/rebase

@rmarinho rmarinho moved this from Todo to Ready To Review in MAUI SDK Ongoing Apr 11, 2025
@github-actions github-actions bot force-pushed the Picker-Focused/Unfocused-Events-Do-Not-Fire-on-Android- branch from e7ddc66 to d626176 Compare April 11, 2025 13:22
@rmarinho rmarinho changed the base branch from main to inflight/current April 11, 2025 13:22
@PureWeen PureWeen merged commit 2f5b2cf into dotnet:inflight/current Apr 16, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from Ready To Review to Done in MAUI SDK Ongoing Apr 16, 2025
@PureWeen
Copy link
Member

I'm reverting this PR because it breaks when using the keyboard

If you use an attached keyboard the picker keeps opening infinitely and then crashes.
What we need to do is get away from using focused/unfocused for pickers and instead look at implementing a new API

#8945

PureWeen added a commit that referenced this pull request Apr 16, 2025
NanthiniMahalingam pushed a commit to NanthiniMahalingam/maui that referenced this pull request Apr 17, 2025
NanthiniMahalingam pushed a commit to NanthiniMahalingam/maui that referenced this pull request Apr 17, 2025
github-actions bot pushed a commit that referenced this pull request Apr 18, 2025
PureWeen added a commit that referenced this pull request Apr 23, 2025
### Description of Change

## .NET MAUI Release Notes - Inflight/Candidate

## What's Changed

### MAUI Product Fixes
* Fix CarouselView layout SR6 regressions by @albyrock87 in
#29035
* Revert "[Android] picker - focus/unfocus events (#28122)" by @PureWeen
in https://github.com/dotnet/maui/pull/1fb5164929
* [Windows] Fixed the flyout content width not being set correctly after
updating to WinUI SDK 1.7 by @Tamilarasan-Paranthaman in
#28996
* [Android] picker - focus/unfocus events by @kubaflo in
#28122
* [XC] add IRootObjectProvider by @StephaneDelcroix in
#28310
* [iOS] Fix for the File.ContentType from MediaPicker not being in valid
MIME format by @SyedAbdulAzeemSF4852 in
#28842
* [Android] Fixed the Incorrect Text Color Applied to Selected Tab in
TabbedPage by @Ahamed-Ali in #28844
* [iOS] Fix FlyoutPage does not respond to changes in the
FlyoutLayoutBehavior property by @devanathan-vaithiyanathan in
#28884
* [Android] Fixed ScalingCanvas.SetBlur not working by
@NirmalKumarYuvaraj in #28911
* [iOS] - Resolved Proper Rendering of Dynamic Header/Footer Updates in
CV2 by @prakashKannanSf3972 in #28641
* [iOS] Fixed the TargetInvocationException Occurs When Selecting
Header/Footer After Changing ItemsLayout in CV2 by @Ahamed-Ali in
#28890
* [Windows] - Fix Visual State Issue with Picker TextColor After
Navigation by @prakashKannanSf3972 in
#28746

### Dependency Updates
* [Windows] Upgrade to Windows App SDK 1.7 by @MartyIX in
#28499

### Testing
* [Testing] Feature Matrix UITest Cases for CollectionView EmptyView
Feature by @NafeelaNazhir in #28679
* Fixed Test case failure in PR 29037 - [2025/04/21] Candidate by
@HarishKumarSF4517 in #29049

**Full Changelog**:
main...inflight/candidate


For more information about inflight process check
https://github.com/dotnet/maui/wiki/Inflight-Branch-Process
@github-actions github-actions bot locked and limited conversation to collaborators May 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Status: Done
7 participants