Skip to content

Recommended directory structure#1

Merged
PhonicUK merged 4 commits intoCubeCoders:mainfrom
kdallas:feature/proposed-folder-structure
Dec 4, 2025
Merged

Recommended directory structure#1
PhonicUK merged 4 commits intoCubeCoders:mainfrom
kdallas:feature/proposed-folder-structure

Conversation

@kdallas
Copy link
Copy Markdown

@kdallas kdallas commented Nov 28, 2025

For your review @PhonicUK

Recommendation for adopting the following directories to match core/production:
Fonts
Images
Scripts
Themes
... the latter "Themes" could later be split into "Less" & "CSS" with minor refactoring.

The image file naming is consistently named, but can also be refactored to accommodate a more defined set of rules.

Fonts\
Images\
Scripts\
Themes\
... the latter "Themes" could later be split into Less & CSS with minor refactoring.
@PhonicUK
Copy link
Copy Markdown
Contributor

Actually I'm thinking more along the lines of having each theme have its own folder, and it can contain whatever fonts/scripts/images it needs?

@kdallas
Copy link
Copy Markdown
Author

kdallas commented Nov 29, 2025

Great idea @PhonicUK -- I like that, each being in a packaged up "bundle" essentially. Then only adding the sub-folders each actually needs (or no sub-folders at all I suppose). You could also have a root info.json or markdown file in each theme folder, instead of placing that in the .css itself. Would you like me to have a crack at this and produce another PR?

@PhonicUK
Copy link
Copy Markdown
Contributor

PhonicUK commented Dec 2, 2025

That would be awesome, yes :) I'll drop this PR. I'd be perfectly happy with an info.json file in each directory with the theme information - but the reason I wanted to put it as metadata in the less/css itself was for backwards compatibility, but I suppose it's not a huge deal.

@PhonicUK PhonicUK merged commit aa83b05 into CubeCoders:main Dec 4, 2025
@kdallas kdallas deleted the feature/proposed-folder-structure branch December 16, 2025 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants