search

eugeneloza / castle-editor-tutorial

Moved to https://gitlab.com/EugeneLoza/castle-editor-tutorial
MIT License
2 stars 0 forks source link

Maybe default editor templates should have pascal units in code subdirectory #7

Open and3md opened 3 years ago

and3md commented 3 years ago

After rereading the tutorial, I noticed that we are telling users to rearrange pascal modules in the default project template. If it is useful in such a simple project, in my opinion we should not require it from users but improve our project templates. If we do that 4.2 tutorial section (Moving units) can be deleted.

After playing some time with editor I agree with @eugeneloza that moving all pascal modules to code directory make navigation in project easier and looks better to me.

@eugeneloza @michaliskambi How about make this change in editor? Maybe this issue should go to CGE repository.

michaliskambi commented 3 years ago

Indeed, I wanted to change it in templates and all examples in CGE. I'll do it ASAP :)

michaliskambi commented 3 years ago

This is now done on CGE side by https://github.com/castle-engine/castle-engine/commit/1f6514f677c51ee4a497405be046dc724b7cd097 . All templates and most examples now have a code/ subdirectory.

I also did somewhat-related https://github.com/castle-engine/castle-engine/commit/e864725ac925f553c792c55ba75e1d8f7317f77f : the templates now contain a sensible .gitignore and README.md that looks good on GitHub out-of-the-box :)

I'm leaving the issue open -- because hopefully @eugeneloza can now simplify the text.

eugeneloza commented 3 years ago

Yeah, many thanks @michaliskambi !