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

Anim replace root #304

Merged
merged 3 commits into from
Feb 23, 2024
Merged

Anim replace root #304

merged 3 commits into from
Feb 23, 2024

Conversation

jayohms
Copy link
Collaborator

@jayohms jayohms commented Feb 23, 2024

Follow up fixes to #286

MichalSznajder and others added 3 commits September 1, 2023 14:58
* main:
  Fix test
  Log whether a visit that has started is a page refresh
  Rename method for consistency with scrolling to anchor proposals
  Use nullable location reference
  Check for window.Turbo and add clarifying comment
  Fix the implementation so it properly works for Turbo 8 refreshes without any flickering. Use Turbo.navigator.location to determine the current location, so it works immediately cold boots and Turbo visits
  Bump minSdkVersion to 26 in the docs
  REPLACE_ROOT presentation - popUpTo to controller.graph.id (inclusive)
  Bump minSdkVersion to 26
  Upgrade to SDK 34 and bump dependencies to latest versions
  Tweak the comment, since not all visit proposals will make it to the app
  After a form redirect to the same page location, don't replace the existing fragment with a new one for a visit
  Use the public strada dependency
  Add the overflow-menu component implementation
  Add the menu component implementation
  Configure the strada form path as a modal screen
  Add the form component implementation
  Intitial work to tie in Strada support with component examples
  Improve how debug logging is enabled and how exceptions are logged

# Conflicts:
#	turbo/src/main/kotlin/dev/hotwire/turbo/nav/TurboNavRule.kt
@jayohms jayohms merged commit 19c3c9c into main Feb 23, 2024
1 check passed
@jayohms jayohms deleted the anim_replace_root branch February 23, 2024 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants