search

qooxdoo / qooxdoo-compiler

Compiler for Qooxdoo, 100% javascript
MIT License
35 stars 23 forks source link

"undefined" output by `qx create` #557

Open johnspackman opened 5 years ago

johnspackman commented 5 years ago

Using qx create blah -I creates Application.js with this header:

/* ************************************************************************

   Copyright: 2019 undefined

   License: MIT license

   Authors: undefined

************************************************************************ */

Also, is it really useful to generate the boot directory? It's entirely optional (the compiler will defer to the standard version) and this adds extra cruft that is the same in many applications

johnspackman commented 5 years ago

Further to this, why do we create a custom Theme? This is how the generator did it, but it would be a easier to understand for first timers and less cruft for experienced users to just set the theme class to qx.theme.Indigo instead of deriving from it.

cboulanger commented 5 years ago

Agree for first-timers, but it is very useful to have a template with a full-fledged theme. So maybe we copy the current one to a separate template and simplify the default? Same thing for boot - it could be part of an "extended" template...

derrell commented 5 years ago

Please don't remove either the template Theme nor template boot directory. Those are extremely useful to beginners! The former has proven over and over to be extremely useful, to avoid painful searching through documentation. The latter will, in my estimation, be widely used to modify the default index.html.

cboulanger commented 4 years ago

Maybe we can add a "minimal" template that omits the cruft. This should be done together as part of #443.