I think there is a bug in `contentColorFor()`, when there are two same background colors in a color ...

# compose

[Kyant]

I think there is a bug in contentColorFor(), when there are two same background colors in a color scheme, it will always return the first content color in both even if we use the second color.

I use contentColorFor(MaterialTheme.colorScheme.surface) but it gives me onBackground.

[Kyant]

IMG_20230429_204732_377.jpg

[Stylianos Gakis]

If background and surface are the exact same color, wouldn’t it make sense that onBackground and onSurface are also the same color?

[Kyant]

No, in my design system, onBackground has less contrast than onSurface in dark theme.

[Kyant]

Like the new text contrast feature in Android 14, they may be different.

[Stylianos Gakis]

That's interesting yeah, I guess it's not that weird of a thing. I'd file a feature request on the issue tracker and link it back here, to see what folks from Google think about this too.

[romainguy]

Considering how this API works I don't see how it could give you the color you want when multiple colors are equal values.

[Kyant]

Well, so do you consider replacing it with other API?
I have to add contentColor for all Surface.

[Kyant]

Or I can add a small value to every channel of background.