Skip to content
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

Fix SearchBar PredictiveBack's animation when a back action was canceled. #1792

Open
wants to merge 1 commit into
base: jb-main
Choose a base branch
from

Conversation

terrakok
Copy link
Member

It is not possible to run a suspend function when the progress flow is canceled because the onBack suspend lambda is canceled as well.

https://android-review.googlesource.com/c/platform/frameworks/support/+/3454206

Release Notes

Fixes - Multiple Platforms

  • Fix SearchBar PredictiveBack's animation when a back action was canceled.

…led.

It is not possible to run a suspend function when the progress flow is canceled because the onBack suspend lambda is canceled as well.

https://android-review.googlesource.com/c/platform/frameworks/support/+/3454206
@terrakok terrakok requested a review from igordmn January 21, 2025 17:46
@igordmn
Copy link
Collaborator

igordmn commented Jan 24, 2025

Please, cherry-pick the commit from AOSP and push it to jb-main (without squashing the PR to preserve the commit).

As for release notes - I assume we don't need them, as we didn't release PredictiveBack yet?

P.S. The previous approach was to not add release notes from AOSP commits, as AOSP itself should has it. Though, we probably need to change that. For example by creating a PR with Release Notes as here, but not squashing it, just pushing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants