Now I’m deeply confused how modifier chaining and ...
# composep
Paul Woitaschek
12/21/2023, 10:51 AMNow I’m deeply confused how modifier chaining and kotlin in general works 🤔
There is a new lint:
https://googlesamples.github.io/android-custom-lint-rules/checks/ModifierFactoryUnreferencedReceiver.md.html
And it says that here the original modifier is unused:
Copy code
private fun Modifier.unicorns() : Modifier = background(Color.Red)s
sindrenm
12/21/2023, 10:55 AMI had this warning some time ago, but it disappeared again. It seems like a bug to me.
background(Color.Red)this.background(Color.Red)thisp
Paul Woitaschek
12/21/2023, 10:56 AMIt’s very weird
s
sindrenm
12/21/2023, 10:56 AMThe warning is probably to warn you if you were to do something like this:
Copy code
private fun Modifier.unicorns() : Modifier = Modifier.background(Color.Red)p
Paul Woitaschek
12/21/2023, 10:56 AMYeah that makes absolute sense
s
sindrenm
12/21/2023, 10:57 AMI'm pretty sure it's a bug in the lint check.
p
Paul Woitaschek
12/21/2023, 10:58 AMYeah me too. Just wondered if there was something fundamentally I don’t understand
s
sindrenm
12/21/2023, 10:59 AMIt could be related to this one, perhaps? https://issuetracker.google.com/issues/187539461
h
Hristijan
12/21/2023, 11:12 AMYou can use .then(//something modifierish)
p
Paul Woitaschek
12/21/2023, 11:12 AMYes that’s what its saying. The question is: Is it necessary
h
Hristijan
12/21/2023, 11:21 AM.then does strict equality check if the modifier is the same otherwise it combines the one that's a receiver and the parameter one, I think that might be an optimization in place that they want us to use
e
eygraber
12/21/2023, 3:16 PMe
efemoney
12/22/2023, 10:14 AMIts a lint bug.
31 Views