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.

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

Fix blank URL_ROOT #4916

Merged
merged 1 commit into from May 15, 2018
Merged

Fix blank URL_ROOT #4916

merged 1 commit into from May 15, 2018

Conversation

ghost
Copy link

@ghost ghost commented Apr 30, 2018

Subject: Fix blank URL_ROOT

Feature or Bugfix

  • Bugfix

Purpose

pathto doesn't expand in non-HTML templates so the workaround involves
expanding it in the HTML file and passing it as a data attribute to
the script.

Detail

  • Bug: Test it with the dirhtml builder and visiting the search page. Because URL_ROOT is blank, the result links are not the expected URL.

Relates

@codecov
Copy link

codecov bot commented Apr 30, 2018

Codecov Report

Merging #4916 into 1.7 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              1.7    #4916   +/-   ##
=======================================
  Coverage   81.96%   81.96%           
=======================================
  Files         282      282           
  Lines       37594    37594           
  Branches     5829     5829           
=======================================
  Hits        30814    30814           
  Misses       5477     5477           
  Partials     1303     1303

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6ad0ad6...a7c6cb0. Read the comment docs.

Copy link
Member

@tk0miya tk0miya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1: But this should be merged into 1.7.x release. Could you rebase this into 1.7 branch please?

@tk0miya tk0miya added this to the 1.7.5 milestone May 2, 2018
@ghost ghost changed the base branch from master to 1.7 May 2, 2018 17:33
pathto doesn't expand in non-HTML templates so the workaround involves
expanding it in the HTML file and passing it as a data attribute to
the script.
@tk0miya tk0miya merged commit b7b1db0 into sphinx-doc:1.7 May 15, 2018
@tk0miya
Copy link
Member

tk0miya commented May 15, 2018

Merged. Thank you for your contribution!

tk0miya added a commit to tk0miya/sphinx that referenced this pull request May 15, 2018
tk0miya added a commit that referenced this pull request May 18, 2018
ix5 added a commit to ix5/isso that referenced this pull request Mar 28, 2022
A few things got refactored in sphinx, especially for 4.x.
Therefore, use provided helpers.

As a reference for the future, always look at what the
"basic" sphinx theme (bundled as `sphinx/themes/basic`)
does.

See:
- sphinx-doc/sphinx@a957d6f710
- sphinx-doc/sphinx#4916
- sphinx-doc/sphinx#4915
- readthedocs/sphinx_rtd_theme#1021
ix5 added a commit to ix5/isso that referenced this pull request Mar 28, 2022
A few things got refactored in sphinx, especially for 4.x.
Therefore, use provided helpers.

As a reference for the future, always look at what the
"basic" sphinx theme (bundled as `sphinx/themes/basic`)
does.

See:
- sphinx-doc/sphinx@a957d6f710
- sphinx-doc/sphinx#4916
- sphinx-doc/sphinx#4915
- readthedocs/sphinx_rtd_theme#1021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants