Remove density checks from M2 swipeable components when deprecated methods are removed [278692145]

Fixed

Bug

Status Update

No update yet.

Description

le...@google.com

created issue #1

Apr 18, 2023 12:34PM

We should remove the pattern we've introduced in swipeable components when the only available constructors is the ones with density.

````
if (scaffoldState.bottomSheetState.density == null) {
val density = LocalDensity.current
SideEffect {
scaffoldState.bottomSheetState.density = density
}
}
```

Comments

ap...@google.com <ap...@google.com> #2Feb 5, 2024 04:52PM

Marked as fixed.

There is a report about it in kotlinlang Slack: https://kotlinlang.slack.com/archives/C01D6HTPATV/p1706005056105739

ap...@google.com <ap...@google.com> #3Feb 5, 2024 04:52PM

Thanks for the report!

pr...@google.com <pr...@google.com> #4Feb 21, 2024 07:32PM

The release notes documentation has been edited to clarify this change in behavior for line height.

To support non-standard text sizes, we encourage users to follow the Material design system and use a different type scale rather than changing the font size directly. Alternatively, users can overwrite the line height like so: style = LocalTextStyle.current.copy(lineHeight = TextUnit.Unspecified), or create a custom Typography entirely.

Issue 278692145

Description

Issue summary

Comments

ap...@google.com <ap...@google.com> #2Feb 5, 2024 04:52PM

ap...@google.com <ap...@google.com> #3Feb 5, 2024 04:52PM

pr...@google.com <pr...@google.com> #4Feb 21, 2024 07:32PM

Add comment

Issue metadata