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

Handle "predictive back" / ActivityTransitionAnimation deprecation in API34 #14558

Open
BrayanDSO opened this issue Oct 19, 2023 · 6 comments · May be fixed by #17594
Open

Handle "predictive back" / ActivityTransitionAnimation deprecation in API34 #14558

BrayanDSO opened this issue Oct 19, 2023 · 6 comments · May be fixed by #17594
Labels
Deprecation Warnings on deprecated API usage Keep Open avoids the stale bot
Milestone

Comments

@BrayanDSO
Copy link
Member

API34 adds new predictive back behavior, which implies that the transition animation overrides for activity stack change animations now need to know whether the animation override is for activity open or close, whereas we override both as a unit without thought

This needs a solution before the deprecated methods go away, likely in the form of plumbing through (via a new parameter) the idea of whether the call site intended to override open, close, or both animations. For backwards compatibility this will likely need to be a Compat interface (or interfaces) to replace the current implementation

Originally posted by @mikehardy in #14556 (comment)

@BrayanDSO
Copy link
Member Author

@lukstbit
Copy link
Member

We also need to fix the onBackPressed() deprecation in #12364 .

@mikehardy mikehardy changed the title Handle ActivityTransitionAnimation deprecation in API34 Handle "predictive back" / ActivityTransitionAnimation deprecation in API34 Oct 19, 2023
@mikehardy
Copy link
Member

mikehardy commented Oct 19, 2023

@lukstbit yep - just had to handle that to pass lint in the stack of PRs, commit here - and examination of that commit (probably altering it or the objects slightly...) should be part of the success criteria to close this issue 9ec94ed

@mikehardy
Copy link
Member

PR #13586 looks like the proper way to go about things - sad (word directed at myself) that the PR has sat so long and accumulated some conflicts! It looks good and should be considered for fixup + merge to fix this for real

@mikehardy
Copy link
Member

Just adding a quick note here that one item has been provisionally handled, so it may not show up in the deprecation scans anymore, but it needs a more thorough handling as it's a temporary patch:

https://github.com/ankidroid/Anki-Android/pull/14712/files#diff-6d9f7ea7bba9cc72513b5078633cee9bdbbd1ea7646114869679f302ad607459R277-R279

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Deprecation Warnings on deprecated API usage Keep Open avoids the stale bot
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants