Add support for Dp.Hairline to Divider [196840810]

Fixed

Bug

Status Update

No update yet.

Description

sa...@squareup.com

created issue #1

Aug 17, 2021 03:29PM

It'd be nice if Divider supported Dp.Hairline. In its current implementation, the divider becomes invisible if Dp.Hairline is used.

Divider(color = Color.LightGray, thickness = Dp.Hairline)

I'm currently working this around by manually calculating 1px in dp:

val density = LocalContext.current.resources.displayMetrics.density
val onePxInDp = (1f / density).dp

I'd also like to call it out that it feels weird to have Dp.Hairline globally available, only to realize that it's up to Composables to manually support it(?). From what I understand, only BorderStroke and Modifier.border account for hairlines in v1.0.1.

Comments

js...@google.com <js...@google.com> Aug 19, 2021 11:47PM

Assigned to lp...@google.com.

lp...@google.com <lp...@google.com> #2Aug 24, 2021 11:37PM

Reassigned to nj...@google.com.

We recently landed stability improvements and bugfixes in aosp/2258262 and will stabilize ModalBottomSheetLayout following a few release cycles. We are still evaluating the future API of BottomSheetScaffold and are expecting it to stay experimental for a bit more while we migrate it to Swipeable V2.

nj...@google.com <nj...@google.com> #3Aug 25, 2021 12:07AM

@jossiwolf is this on track for 1.6 or moved out to 1.7?

ap...@google.com <ap...@google.com> #4Aug 26, 2021 02:14PM

Marked as fixed.

Currently slated for 1.7

Issue 196840810

Description

Issue summary

Comments

js...@google.com <js...@google.com> Aug 19, 2021 11:47PM

lp...@google.com <lp...@google.com> #2Aug 24, 2021 11:37PM

nj...@google.com <nj...@google.com> #3Aug 25, 2021 12:07AM

ap...@google.com <ap...@google.com> #4Aug 26, 2021 02:14PM

Add comment

Issue metadata