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

Downgrade navigation library to 2.6.0, upgrade deps #22

Merged
merged 2 commits into from Oct 26, 2023

Conversation

jocmp
Copy link
Collaborator

@jocmp jocmp commented Oct 26, 2023

  • Addresses an issue where navigation causes unexpected slide/scale animation when using compose navigation 2.7 and above. The solution is to downgrade to 2.6 and the problem goes away. See https://issuetracker.google.com/issues/297258205 for details.
  • Sets the video's current time from the video player position listener so that the time will be set during instantiation instead of on resume. This avoids any judder when seeking to the video's time on orientation change.

Relates to #20

Addresses an issue where navigation causes unexpected
slide/scale  animation when using compose navigation 2.7
and above. The solution is to downgrade to 2.6 and the problem
goes away.
Sets the video's current time from the video player position listener
so that the time will be set during instantiation instead of on resume.
This avoids any judder when seeking to the video's time on orientation
change.
@jocmp jocmp merged commit 832ca0a into main Oct 26, 2023
1 check passed
@jocmp jocmp deleted the jc/routing-animation branch October 26, 2023 21:35
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.

None yet

1 participant