Skip to content
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

Use ejs templates in browse JS #127

Closed
wants to merge 4 commits into from
Closed

Conversation

jfirebaugh
Copy link
Member

There are a few other JS-related cleanups in this PR, but
the main commit replaces the tedious manual DOM manipulation
with EJS templates.

EJS is sort of the lowest common denominator JS templating
language. It would be fairly easy to port to mustache or
something if anyone has a strong preference. I've purposefully
kept the templates pretty much logic-free.

@tomhughes
Copy link
Member

I'm all for something along these lines, so I'd love to hear what other people think both in general and about what JS templating system we should use.

@tomhughes
Copy link
Member

Nobody objected to this, so I merged it, mostly because I wanted it for my work on the notes code ;-)

@tomhughes tomhughes closed this Oct 13, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Refactoring, or things that work but could be done better
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants