purescript-contrib / purescript-css

A clean, type-safe library for describing, manipulating and rendering CSS
Apache License 2.0
105 stars 39 forks source link

Remove usages of Data.Tuple.Nested. #44

Closed chexxor closed 3 years ago

chexxor commented 7 years ago

Data.Tuple.Nested moved to meaning extensible tuples, which implies ending the Tuple in Unit. This added Unit causes an extra space to be added to a tuple-backed CSS rule, like animation. In addition, we aren't using tuple-combinators, which extensible tuples enables.

See discussion: https://github.com/slamdata/purescript-css/pull/43#issuecomment-257904343

Note that this patch isn't DRY - I reimplemented tuple4 several times. What's the preferred way to handle that?

safareli commented 7 years ago

Maybe just use do something like this a `Tuple` b `Tuple` c

JordanMartinez commented 3 years ago

Superceded by #135.