themeteorchef / base

A starting point for Meteor apps.
http://themeteorchef.com/base
MIT License
689 stars 253 forks source link

Subdirectory for components, containers & pages #215

Open rgstephens opened 7 years ago

rgstephens commented 7 years ago

I'd like to see a documents subdirectory for ui/components, containers & pages in the same way the api has a documents subdirectory. When I create a new project with Base and starting making new collections and related UI components, I create a subdirectory for each new collection under both api and the ui directories.

arthurkirsz commented 7 years ago

Make sense to me. When the app grows, I always struggle organising with Base

rgstephens commented 7 years ago

I submitted pull request #216 with the changes along with a related request, pull #217, to change the filenames for easier extension of Base.

arthurkirsz commented 7 years ago

Had a quick look. I like it but Ryan might not merge the PRs. The current structure is very clear for beginners whereas the new one could make less sense.what do you think Ryan ?

themeteorchef commented 7 years ago

Hi @rgstephens @arthurkirsz. Agree with Arthur here. I do prefer this more personally, but I want to keep things as close to the Meteor Guide/simple as possible. Something like an extra directory may seem like a silly thing to split hairs over, but—especially for beginners—it introduces the added question of "do I have to do it this way?"

Going to think about this and flag it as a consideration for Base 5.0. Thanks for submitting this, Greg :)