outwatch / purescript-outwatch

A functional and reactive UI framework based on Rx and VirtualDom
https://outwatch.github.io/?lang=purescript
Apache License 2.0
34 stars 5 forks source link

about naming, consistency, name clashes, module organisation #7

Open rvion opened 7 years ago

rvion commented 7 years ago

I love OutWatch's overall design, but have some issues with consistency, modules organisation, and naming clashes (I saw the latests re-organisation commits, but I think there is more to discuss)

Purescript

names like div are really annoying to use because they clash with prelude. name like tpe are also annoying to use because they are harder to remember

would it make sense to follow lucid design and prefix or postfix everything with an _ ?

it could also help adding back an global OutWatch that reexport most useful functions

(to see some more lucid examples, github link is here: https://github.com/chrisdone/lucid)

LukaJCB commented 7 years ago

Hey @rvion thanks a lot for bringing this up! :)

I can definitely see the issues with the naming clashes in prelude and I'm not happy with it either. However I'm not a hundred % convinced yet, that this is the answer. I can't really point to exactly why, but to me it's not pleasing solution yet.

Maybe, though, we could support both? Then you can import the module you like and go with that if you value consistency.

rvion commented 7 years ago

indeed, that's what I've been experimenting with, I'll share that very soon :)