Rename state parameter on Modifier.sharedElement to match same naming as Modifier.sharedBounds [375351468]

Fixed

Feature Request

Status Update

No update yet.

Description

ri...@google.com

created issue #1

Oct 24, 2024 10:17AM

Modifier.sharedElement state parameter is named differently from Modifier.sharedBounds, making switching between the two a bit more cumbersome than needed.

    override fun Modifier.sharedElement(
        state: SharedContentState,
        animatedVisibilityScope: AnimatedVisibilityScope,

and Modifier.sharedBounds:

    override fun Modifier.sharedBounds(
        sharedContentState: SharedContentState,

It is desirable to have these consistent names between the two.

Comments

ap...@google.com <ap...@google.com> #2Oct 25, 2024 06:24PM

Marked as fixed.

Thanks! This looks like a bug, thank you for reporting!

na...@google.com <na...@google.com> #3Nov 13, 2024 06:19PM

I would also recommend setting the value to 10%. When you do that, the behavior is wild.

Dragging up to 10% or between 50% and 90% will cause it to snap back to the start anchor.
Dragging between 10% and 50% or past 90% will cause it to snap to the end anchor.

Issue 375351468

Description

Issue summary

Comments

ap...@google.com <ap...@google.com> #2Oct 25, 2024 06:24PM

na...@google.com <na...@google.com> #3Nov 13, 2024 06:19PM

Add comment

Issue metadata