-
Notifications
You must be signed in to change notification settings - Fork 917
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
layouts: split footer partial in several partials #1500
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general, I'm fine with these changes. Directory layouts/partials
contains a lot of files already, therefore I would propose to create a new directory layouts/partials/footer
and put the newly introduced files foooter-xxx.html
into that directory.
635a733
to
bd937f7
Compare
That’s indeed a good idea! |
bd937f7
to
0e07183
Compare
0e07183
to
75de4a3
Compare
That's just what I was looking for, I hope it gets merged soon. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM now.
Hi @yann-soubeyrand: thanks for this. I'm eager to look at it once the BS v5.2 upgrade is done and the next release of Docsy is out. Could you resolve conflicts in the meantime? Thanks! |
75de4a3
to
1f6fc75
Compare
Hi @chalin, it’s done 😉 |
3f4f6f1
to
c93ee1c
Compare
c93ee1c
to
4233cc7
Compare
4233cc7
to
e3239a4
Compare
9e33776
to
976832d
Compare
This allows to more easily override some parts of the footer. Signed-off-by: Yann Soubeyrand <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! ✨
This allows to more easily override some parts of the footer.
Related: