purescript-cookbook icon indicating copy to clipboard operation
purescript-cookbook copied to clipboard

Simplify ComponentsHalogenHooks

Open milesfrain opened this issue 4 years ago • 1 comments

I was initially confused about how ButtonMessage was being used, and then realized that the component's output value is simply ignored by the parent when incrementing toggle count. So this can be replaced with Unit. Don't know whether this makes things more or less confusing. We could alternatively go with an Output alias and some more comments.

milesfrain avatar Dec 07 '20 23:12 milesfrain

Nah, that's fine. Using unit is better.

JordanMartinez avatar Dec 08 '20 03:12 JordanMartinez