Folder "views" in Custom Themes #4238

Closed
opened 2026-02-05 08:19:07 +03:00 by OVERLORD · 4 comments
Owner

Originally created by @ikabod on GitHub (Sep 26, 2023).

Describe the feature you'd like

Hi,

you have the folder "icons" in the custom themes folder to override icons. You have the folder "lang" to override translation texts.

Why not a folder named "views" for all view overrides?

Thanks,
Peter

Describe the benefits this would bring to existing BookStack users

This would make the folder structure more cleaner, especially if somebody (like me) have some other folders too, e.g. "php" for own class files and "js" and "css" folder.

Can the goal of this request already be achieved via other means?

If you are a .httaccess expert, may be you can provide an example to move an existing folder "/themes/my-theme/common/" to "/themes/my-theme/views/common/"?

Have you searched for an existing open/closed issue?

  • I have searched for existing issues and none cover my fundamental request

How long have you been using BookStack?

Under 3 months

Additional context

No response

Originally created by @ikabod on GitHub (Sep 26, 2023). ### Describe the feature you'd like Hi, you have the folder "icons" in the custom themes folder to override icons. You have the folder "lang" to override translation texts. Why not a folder named "views" for all view overrides? Thanks, Peter ### Describe the benefits this would bring to existing BookStack users This would make the folder structure more cleaner, especially if somebody (like me) have some other folders too, e.g. "php" for own class files and "js" and "css" folder. ### Can the goal of this request already be achieved via other means? If you are a .httaccess expert, may be you can provide an example to move an existing folder "/themes/my-theme/common/" to "/themes/my-theme/views/common/"? ### Have you searched for an existing open/closed issue? - [X] I have searched for existing issues and none cover my fundamental request ### How long have you been using BookStack? Under 3 months ### Additional context _No response_
OVERLORD added the 🔨 Feature Request label 2026-02-05 08:19:07 +03:00
Author
Owner

@ssddanbrown commented on GitHub (Sep 26, 2023):

Why not a folder named "views" for all view overrides?

Because the system was created just for views initially, then extended to the other content types.
Don't really want to change things for existing users.
Could run both options (Check both paths) but I'm not too keen on adding adding ambiguity to the theme system, for non-back-compat reasons, and I don't assure that folders/paths will always mirror the resources folder, things may change.

@ssddanbrown commented on GitHub (Sep 26, 2023): > Why not a folder named "views" for all view overrides? Because the system was created just for views initially, then extended to the other content types. Don't really want to change things for existing users. Could run both options (Check both paths) but I'm not too keen on adding adding ambiguity to the theme system, for non-back-compat reasons, and I don't assure that folders/paths will always mirror the resources folder, things may change.
Author
Owner

@ikabod commented on GitHub (Sep 26, 2023):

Suggestion: A script, which runs on the next update only and moves all folders which exists in "/resources/views/" into the new "/themes/my-theme/views/" folder.

@ikabod commented on GitHub (Sep 26, 2023): Suggestion: A script, which runs on the next update only and moves all folders which exists in "/resources/views/" into the new "/themes/my-theme/views/" folder.
Author
Owner

@ssddanbrown commented on GitHub (Sep 26, 2023):

I'm not really sure what that achieves, at cost to maintaining. I'd rather users only have the folders they need.
Could always script that for your own instance if desired.

@ssddanbrown commented on GitHub (Sep 26, 2023): I'm not really sure what that achieves, at cost to maintaining. I'd rather users only have the folders they need. Could always script that for your own instance if desired.
Author
Owner

@ssddanbrown commented on GitHub (Oct 26, 2023):

I'll proceed to close this off as per my reasoning above.

@ssddanbrown commented on GitHub (Oct 26, 2023): I'll proceed to close this off as per my reasoning above.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#4238