✨ NEW: Add logo_only theme option

[djangoliv]

Support for logo_only option.

Regars

[welcome]

Thanks for submitting your first pull request! You are awesome! 🤗

If you haven't done so already, check out EBP's Code of Conduct and our Contributing Guide, as this will greatly help the review process.

Welcome to the EBP community! 🎉

[choldgraf]

This looks good to me, with a couple extra requests:

• could you add documentation of this new feature somewhere around https://sphinx-book-theme.readthedocs.io/en/latest/configure.html#customize-the-logo-title-and-favicon ?
• could you install pre-commit and run pre-commit install before you make your next commit? This will ensure that the pre-commit tests pass

[djangoliv]

@choldgraf done:

pre-commit ok:

Check JSON...........................................(no files to check)Skipped
Check Yaml...........................................(no files to check)Skipped
Fix End of Files.........................................................Passed
Trim Trailing Whitespace.................................................Passed
flake8...............................................(no files to check)Skipped
black................................................(no files to check)Skipped
check-manifest...........................................................Passed
Compile Web Assets...................................(no files to check)Skipped

[choldgraf]

ahh I figured out what was going on - we needed to update the conf in src/jinja not the one in sphinx_book_theme

[choldgraf]

caught a little bug in the docs (we need this option provided under html_theme_options). Also added a test. Should be working now

[welcome]

Congrats on your first merged pull request in this project! 🎉


Thank you for contributing, we are very proud of you! ❤️