Skip to content
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.

Sign up for GitHub

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

Sphinx 6: Use logo_url instead of logo #1405

Merged
merged 5 commits into from
Jan 17, 2023
Merged

Sphinx 6: Use logo_url instead of logo #1405

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
2142c6d
Sphinx 6: Use logo_url instead of logo
benjaoming Jan 11, 2023
1e11b3a
Handle removal of favicon template context + refactor display of logo…
benjaoming Jan 11, 2023
19aaba5
Adds a favicon to the demo
benjaoming Jan 12, 2023
f355651
Import condensed version logic @agjohnson
benjaoming Jan 13, 2023
0739023
Update sphinx_rtd_theme/layout.html
benjaoming Jan 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions docs/conf.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@

html_logo = "demo/static/logo-wordmark-light.svg"
html_show_sourcelink = True
html_favicon = "demo/static/favicon.ico"

htmlhelp_basename = slug

Expand Down
Binary file added docs/demo/static/favicon.ico
View file
Open in desktop
Binary file not shown.
39 changes: 25 additions & 14 deletions sphinx_rtd_theme/layout.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,11 +41,13 @@
{%- endfor -%}

{#- FAVICON #}
{%- if favicon %}
{%- if sphinx_version_info < (4, 0) -%}
<link rel="shortcut icon" href="{{ pathto('_static/' + favicon, 1) }}"/>
{%- else %}
<link rel="shortcut icon" href="{{ favicon_url }}"/>
{%- if sphinx_version_info < (4, 0) -%}
{%- if favicon %}
<link rel="shortcut icon" href="{{ pathto('_static/' + favicon, 1) }}"/>
{%- endif %}
{%- else %}
{%- if favicon_url %}
<link rel="shortcut icon" href="{{ favicon_url }}"/>
{%- endif %}
{%- endif -%}

Expand Down Expand Up @@ -133,23 +135,32 @@
<div class="wy-side-nav-search" {% if theme_style_nav_header_background %} style="background: {{theme_style_nav_header_background}}" {% endif %}>
{%- block sidebartitle %}

{%- if logo and theme_logo_only %}
{%- if ( logo or logo_url ) and theme_logo_only %}

{%- if sphinx_version_info < (4, 0) -%}
{# the logo helper function was removed in Sphinx 6 and deprecated since Sphinx 4 #}
{# the master_doc variable was renamed to root_doc in Sphinx 4 (master_doc still exists in later Sphinx versions) #}
{%- if logo %}
<a href="{{ pathto(master_doc) }}">
<img src="{{ pathto('_static/' + logo, 1) }}" class="logo" alt="{{ _('Logo') }}"/>
</a>
{%- endif %}
{# Behavior since Sphinx 4 #}
{%- else %}
<a href="{{ pathto(master_doc) }}" class="icon icon-home"> {{ project }}
{% if logo_url %}
<a href="{{ pathto(root_doc) }}">
<img src="{{ logo_url }}" class="logo" alt="{{ _('Logo') }}"/>
</a>
{%- endif %}
{%- endif %}

{%- if logo %}
{#- Not strictly valid HTML, but it's the only way to display/scale
it properly, without weird scripting or heaps of work
#}
{%- else %}
{%- if sphinx_version_info < (4, 0) -%}
<img src="{{ pathto('_static/' + logo, 1) }}" class="logo" alt="{{ _('Logo') }}"/>
<a href="{{ pathto(master_doc) }}" class="icon icon-home"> {{ project }}</a>
{%- else %}
<img src="{{ logo_url }}" class="logo" alt="{{ _('Logo') }}"/>
<a href="{{ pathto(root_doc) }}" class="icon icon-home"> {{ project }}</a>
{%- endif %}
{%- endif %}
</a>

{%- if theme_display_version %}
{%- set nav_version = version %}
Expand Down