-
Notifications
You must be signed in to change notification settings - Fork 91
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
Is Jade Necessary? #16
Comments
Hey. This is a good point, it does seem superfluous upon reflection. I'll swap it out for standard HTML template so that this aspect is simplified and easier to understand for people without Jade knowledge. Thank you for your input! |
Awesome. And thanks for putting this together! |
Please check out this PR. I chose the Let me know your thoughts. |
This PR has now been closed. I am going to close this issue but if you notice anything else feel free to open a new one! :) |
Don't get me wrong. I love Jade (and HAML and Slim), but you'll only even be using it for the entry point on the server so it'll never be larger than this:
https://github.com/GordyD/3ree/blob/master/server/views/index.jade
It's not really part of this stack and seems unnecessary to include it just for this one view. Besides you're using JSX (HTML) on the react side, so now you've got multiple syntax for rendering.
Just an opinion though!
The text was updated successfully, but these errors were encountered: