mikesol / purescript-deku

A PureScript web UI framework
https://purescript-deku.surge.sh/
Apache License 2.0
123 stars 12 forks source link

Adds inline pursx #59

Closed mikesol closed 1 year ago

mikesol commented 1 year ago

Looks super ugly without VTAs, but pretty sweet that this is actually possible! One step closer to JSX-like syntax :-)

MonoidMusician commented 1 year ago

Looks fine. Two small questions:

mikesol commented 1 year ago

Looks fine. Two small questions:

  • is Append ll "a" lll just to get fresh variables?
  • the (Proxy ll /\ Proxy html /\ { | r }) setup is a little weird to my eyes (as opposed to using separate type variables), but there's nothing wrong with it really
MonoidMusician commented 1 year ago

I suggest we standardize these operators instead of having custom functions for each attribute: https://github.com/MonoidMusician/blog/blob/25a303060e6f26cb27b6101b03b49abefcd33b0c/PureScript/src/FRP/Deku.purs#L14-L28

mikesol commented 1 year ago

I suggest we standardize these operators instead of having custom functions for each attribute: https://github.com/MonoidMusician/blog/blob/25a303060e6f26cb27b6101b03b49abefcd33b0c/PureScript/src/FRP/Deku.purs#L14-L28

Thanks for the suggestion! Pushed as eb6ac8a.