Wedding Static Micro-Website

I am a bit embarrassed about the code quality, but it was a quick and dirty job. I figure I may get good feedback on improvements and/or it may be useful to others for their nuptials…




You shouldn’t feel embarrassed, It’s everything fine and organized.
I have a few nitpicks, but they may be only personal preferences, and you may disagree.

So, in my opinion:

  • Avoid exposing all (..) in a module
  • Avoid importing all in a module - e.g. Data module
  • Main init doesn’t seem to be heavy, so I wouldn’t use unit parameter there.
  • I avoid using the magic record constructor function outside decoders. Instead, I prefer to be explicit about the record fields.
  • Also, I like to add type signatures to almost all of my let-in definitions
1 Like

Thanks very much for that feedback! I also feel like function application/composition operators could also clean things up a lot, and there are other improvements needed I listed in the README file

1 Like

What a lovely project! I really like the visual design. It looks very tasteful and refined. :slight_smile:

The timetable looks perhaps a bit misaligned on mobile, but that might just be me.

Great stuff!

1 Like

Worth noting - most of these can be auto fixed by elm-review.

1 Like

This topic was automatically closed 10 days after the last reply. New replies are no longer allowed.