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

Move from utml to Jade #1167

Closed
strugee opened this Issue May 25, 2016 · 4 comments

Comments

Projects
None yet
1 participant
@strugee
Member

strugee commented May 25, 2016

We're moving from utml to Jade, because Jade is popular and people like using it and also utml is super busted[1].

This work is already happening in the utml-to-jade branch. Things that need to happen after this is mostly complete:

  • Make sure there aren't any lingering class="" instances
  • Ditto for id=""
  • Clean up div.classname to be just .classname
  • Ditto for div.#idname
  • Make sure there aren't any lingering TODOs
  • Evaluate if includes should be changed to mixins (grep around for - var)

[1]: It doesn't work on Express 3.x/4.x, and when I went to the source to fix it up, I saw synchronous filesystem reads and said "screw it, it's not worth it".

@strugee

This comment has been minimized.

Show comment
Hide comment
@strugee

strugee Jun 1, 2016

Member

This is actually almost done. However, it's blocked because of pugjs/pug#2409

Member

strugee commented Jun 1, 2016

This is actually almost done. However, it's blocked because of pugjs/pug#2409

@strugee strugee modified the milestones: 1.1, 1.0.0 Jun 24, 2016

@strugee

This comment has been minimized.

Show comment
Hide comment
@strugee

strugee Jul 19, 2016

Member

It's been quite a while since I updated this - that Pug bug got fixed, but the current problem is that the frontend JS doesn't update correctly because of logic having to do with utml partials.

Member

strugee commented Jul 19, 2016

It's been quite a while since I updated this - that Pug bug got fixed, but the current problem is that the frontend JS doesn't update correctly because of logic having to do with utml partials.

@strugee

This comment has been minimized.

Show comment
Hide comment
@strugee

strugee Aug 11, 2016

Member

Aaand this is actually done! I've put out a call for testing if anyone would like to help out the project and try this on their Pump node.

Member

strugee commented Aug 11, 2016

Aaand this is actually done! I've put out a call for testing if anyone would like to help out the project and try this on their Pump node.

@strugee

This comment has been minimized.

Show comment
Hide comment
@strugee

strugee Sep 12, 2016

Member

Closing as this has been in master for a couple days now

Member

strugee commented Sep 12, 2016

Closing as this has been in master for a couple days now

@strugee strugee closed this Sep 12, 2016

@strugee strugee modified the milestones: 2.0, Future Oct 31, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment