Rename "props" to "styles" or something else

[dongepulango]

The name props that is pass to style={} is very confusing with the regular React props.value. I'm sure I'm not the only one thinking this. I've seen some tutorials that says it's confusing to them as well.

I'm sure there is a reason why this is called props, but for me it's not helping. styles or animations is my suggestions.

[aleclarson]

You can name it whatever you like, no?

const styles = useSpring({ ... })
const [styles, update] = useSpring(() => ({ ... }))

[dongepulango]

You can, but would it be better if you don't have to?

[aleclarson]

I don't understand. Can you give an example that would be easier?

[dongepulango]

I mean there ^

[aleclarson]

Ahh, you didn't mention useTransition specifically.

[dongepulango]

Sorry about that. I know you can rename them props: styles but I just find it confusing sometimes, as well as other people in tutorials.

[drcmda]

the thing is, they're not styles, they can be, but essentially you're just interpolating arbitrary props, which you then cast into styles/attributes/etc. Curiously, react-motion naming it "styles" was often criticized for that reason, people were saying it was confusing as well: chenglou/react-motion#538 😄

but you're right, the api isn't optimal. sometimes i think that going for an object was a mistake over:

transitions.map(([visible, key, styles]) => ...

upside: naming, downside: you have to remember the correct order.

we also had some other ideas, see: #665

[dongepulango]

Thanks for the explanation.

[drcmda]

wait, you're too quick : D this is a totally valid point, we can sure discuss how we can maybe solve it in a better way. feel free to give us your suggestions. do you like the array destructure for instance?