-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add the ability to nest layouts #2886
Comments
The layout's currently get rendered from https://github.com/octobercms/october/blob/master/modules/cms/classes/Controller.php#L373 so use that as a starting point to figure out how to implement nested layouts. |
This is going to need some clear advantage over the following: root-layout.htm
nav-layout.htm
titled-layout.htm
about.htm
|
@daftspunk For the I also can't tell where the |
This issue will be closed and archived in 3 days, as there has been no activity in the last 30 days. If this issue is still relevant or you would like to see action on it, please respond and we will get the ball rolling. |
This issue will be closed and archived in 3 days, as there has been no activity in the last 30 days. If this issue is still relevant or you would like to see action on it, please respond and we will get the ball rolling. |
This issue will be closed and archived in 3 days, as there has been no activity in the last 30 days. If this issue is still relevant or you would like to see it actioned, please respond and we will re-open this issue. |
@LukeTowers were you planning on implementing this? |
@bennothommo yes, at some point |
This issue will be closed and archived in 3 days, as there has been no activity in the last 30 days. |
This issue will be closed and archived in 3 days, as there has been no activity in the last 30 days.
|
This issue will be closed and archived in 3 days, as there has been no activity in the last 30 days. |
This issue will be closed and archived in 3 days, as there has been no activity in the last 30 days. |
This issue will be closed and archived in 3 days, as there has been no activity in the last 60 days. |
This issue will be closed and archived in 3 days, as there has been no activity in the last 60 days. |
This issue will be closed and archived in 3 days, as there has been no activity in the last 60 days. |
This issue will be closed and archived in 3 days, as there has been no activity in the last 60 days. |
This issue will be closed and archived in 3 days, as there has been no activity in the last 60 days. |
It is an age-old DRY vs WET debate. This adds significant overhead to the experience of October so as mentioned we'd need to see a clear advantage to justify it. Thanks for the suggestion anyhow! |
It would be nice if layouts in October could be nested so that you could pick your injection point! An example would be...
root-layout.htm
nav-layout.htm
titled-layout.htm
about.htm
Using this syntax allows you to maintain your tag pairs and pick your injection point for the content of a page. i.e. the homepage would set it's layout to the
nav-layout
whereas the about pages sets its layout to thetitled-layout
The text was updated successfully, but these errors were encountered: