Spacing looks wrong.

@ThibG The Collapsable was basically putting style/appearance in the logic instead of the CSS, in a much verbose and less understandable way. Also, now changing margins happens in one place, which makes changing default theme/theming easier.

@Gargron My reasoning was that as the CW is part of the toot, the margin between the CW field and the toot field should be thinner. I can make both margin the same size, if you like it better this way.

@Gargron up? Do you still think the spacing needs change? Are there other issues with that PR?

Yes, I do think the spacing needs change.

I also haven't seen how the animation looks now. I'm not so sure: spring-based animation is generally more interesting to look at than CSS transitions. I do not agree with the arguments about style in the markup, we use Motion throughout the application so it's more consistent for us to have that than CSS transitions.

The problem I had with height of the element in the markup is that I couldn’t modify the size of the spacing under the CW easily with my custom CSS extension (I assume it’s the same for themes).

Here’s how it looks now (before/after):

I used the same 10px spacing that’s used between the toot field and the toot button.

I also kept the CSS animation, since I didn’t notice any difference visible difference with the Spring one, and for the easier theming of margins.