search

rpgtkoolmv / corescript

http://www.rpgmakerweb.com/products/programs/rpg-maker-mv
MIT License
311 stars 75 forks source link

Why not es6 class? #211

Closed carnoxen closed 4 years ago

carnoxen commented 4 years ago

this engine is full of prototype-things, so it's hard to read. nw.js supports the class declaration, then why it wasn't written by es6 class?

DKPlugins commented 4 years ago

To support older browsers when porting. In order not to use transpilers.

carnoxen commented 4 years ago

To support older browsers when porting. In order not to use transpilers.

Is it necessary to consider older browser? Class support is available on all browser except IE.

inc0der commented 4 years ago

Is it necessary to consider older browser? Class support is available on all browser except IE.

Not really necessary, and the devs could've included a transpiler in the editor so when you export you have the option of choosing. Heck, this repo could change to ES6 classes and provide transpiled files so we have a choice. Even you could clone this repo and change all them to ES6 classes and do it yourself if you want to.

carnoxen commented 4 years ago

Is it necessary to consider older browser? Class support is available on all browser except IE.

Not really necessary, and the devs could've included a transpiler in the editor so when you export you have the option of choosing. Heck, this repo could change to ES6 classes and provide transpiled files so we have a choice. Even you could clone this repo and change all them to ES6 classes and do it yourself if you want to.

I didn’t try using transpiler... but I will.