Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

The great switch to Mustache #118

Closed
2 of 17 tasks
neongreen opened this issue Jul 31, 2016 · 5 comments
Closed
2 of 17 tasks

The great switch to Mustache #118

neongreen opened this issue Jul 31, 2016 · 5 comments

Comments

@neongreen
Copy link
Member

neongreen commented Jul 31, 2016

HTML shall no longer be generated with code! (Or rather, it'll be generated with Mustache-like code that will be interpreted in real time! But also cached!)

  • Restore ids (item-arhg34r for items, trait-shh339 for traits)
  • Review all TODOs and delete ones that are fixed by this switch
  • Move all CSS to relevant widgets
  • Make sure that mustache.js is used (or delete it)
  • Remove the lucid dependency (if it's unused)
  • Re-validate generated HTML (see The app generates invalid HTML #7)
  • Get rid of JS.hs
  • Use packunused on the project
  • Split widgets between haskell/, admin/, utils/
  • Describe how the whole system works in README
  • Error out on unknown section names (sections = HTML/Description/etc)
  • Report accurate parser error positions
  • Warn when two partials have the same name
  • Try to avoid Emacs symlink warnings by using Broken symlinks break directory watching haskell-fswatch/hfsnotify#66 (comment)
  • cache generated JS and CSS

Also would be nice to:

  • Use LESS or something instead of CSS
  • Validate generated JS
@neongreen neongreen added this to the public release milestone Jul 31, 2016
@neongreen
Copy link
Member Author

I'm going to abort this task (because Mustache is a sucky HTML templating language and I want something like Pug instead), but for now I'll keep it open because the list can still be useful.

@neongreen
Copy link
Member Author

I want something like Pug instead

We ended up rewriting the frontend in React :trollface:

@chshersh
Copy link
Contributor

@neongreen @sectore I see that frontend is written in PureScript. What templating engine are you using in PureScript?

@sectore
Copy link
Contributor

sectore commented Jul 11, 2018

@chshersh It's purescript-smolder, which comes with Pux.

@neongreen neongreen removed this from the public release milestone Sep 1, 2018
@neongreen
Copy link
Member Author

This was a rather bad idea in retrospect.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants