cleaver icon indicating copy to clipboard operation
cleaver copied to clipboard

Possible enhancement: rewrite to es6

Open sudodoki opened this issue 8 years ago • 4 comments

Just throwing it out there. Maybe, someone w/ much free time on their hands might do that. Incorporating fixes for all possible design issues and pull request… 🤔

sudodoki avatar Jan 11 '17 14:01 sudodoki

What would be the benefit of this?

hhzl avatar Jan 25 '17 13:01 hhzl

@hhzl no visible benefit apart from shorter syntaxes at places and excuse to rewrite pieces ones doesn't like

sudodoki avatar Jan 25 '17 16:01 sudodoki

Maybe template strings could be put to good use making the code easier to read and maintain...

https://developers.google.com/web/updates/2015/01/ES6-Template-Strings

On 1/25/17, Джон, просто Джон [email protected] wrote:

@hhzl no visible benefit apart from shorter syntaxes at places and excuse to rewrite pieces ones doesn't like

-- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/jdan/cleaver/issues/164#issuecomment-275161189

hhzl avatar Jan 25 '17 17:01 hhzl

I mean only for the part generating the result html presentation file, not within the html presentation. That should probably better stay JavaScript 5.

That would mean replacing the mustache template system, http://mustache.github.io/. Not sure if it is worth the effort.

The issue could then be named

Replace the mustache template system with ES6 template strings.

hhzl avatar Jan 25 '17 17:01 hhzl