-
Notifications
You must be signed in to change notification settings - Fork 26
Apply custom footer and custom MkDocs Theme #722
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
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.
Some comments about the footer!
|
We have to adapt this PR to download https://github.com/jolicode/oss-footer/blob/main/snippet-joli-footer.html at build time instead. |
|
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.
Pull Request Overview
This PR refactors the MkDocs footer implementation to use external assets from the jolicode/oss-theme repository. The copyright notice is removed from the MkDocs configuration and replaced with a dynamic footer that includes custom branding and styling.
- Adds a new
fetch_assets()task to download external CSS and footer HTML from the jolicode/oss-theme repository - Removes hardcoded copyright from mkdocs.yml and implements a custom footer partial
- Updates both build and serve tasks to fetch assets before execution
Reviewed Changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tools/mkdocs/castor.php | Adds fetch_assets() function to download external assets and integrates it into build/serve tasks |
| tools/mkdocs/mkdocs.yml | Removes copyright field and adds reference to new jolicode.css stylesheet |
| tools/mkdocs/overrides/main.html | Adds footer block that includes the jolicode-footer.html partial |
| tools/mkdocs/.gitignore | Ignores the dynamically downloaded jolicode-footer.html file |
| .gitignore | Ignores the dynamically downloaded jolicode.css file |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
6e8815f to
f76e3bc
Compare
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.
Thanks for the work 💛
|
@jonathanlevaillant 👋❤️😄 |
|
Hey @tigitz ! 👋💛😄 |
No description provided.