-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Exterminate Jade #428
Comments
I prefer Jade, but I can go with whatever the majority prefers. 🍶 |
what would be the wins here? or just a rewrite to rewrite? |
From what I can see:
In response to my own points:
|
I think the content is the king (built upon JS documentation), the build process should be saved as much as possible for content, not markup generation. The rest should be reserved for optimizing content/markup size. |
It could be worthwhile streamlining it. I'm not familiar with the site setup but I've noticed there's a handlebars template there too. At a higher level, if we have a blessed way of writing Marionette, such as what template engine to prefer by default, then we'd be best writing the site to follow that, in my opinion. |
Personally, I hate writing Jade. It's hard for me to read and given that HTML isn't a particularly difficult language to write, I feel like Jade is unnecessary. It's just another step in our insanely long build process.
How would everybody else feel about removing Jade and moving to using plain old HTML and maybe Handlebars or Underscore templates?
NOTE: These are obviously personal opinions. If everyone else prefers Jade I won't be offended. 😄
/cc @jdaudier @peterblazejewicz @scott-w @paulfalgout
The text was updated successfully, but these errors were encountered: