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 "Edit on Github" links #577

Merged
merged 3 commits into from
Dec 4, 2024
Merged

Fix "Edit on Github" links #577

merged 3 commits into from
Dec 4, 2024

Conversation

santisoler
Copy link
Member

@santisoler santisoler commented Nov 29, 2024

Fix the "Edit on Github" links by removing the _templates/breadcrumbs.html file: Sphinx and the RTD theme can generate those links if the docs are correctly configured. Add "conf_py_path" to the html_context dictionary in Sphinx configuration file, so the links to the source files work correctly. Minor updates on the configuration file to avoid some warnings: remove the language=None and html_theme_path.

Alternative solution to #575

Fix the "Edit on Github" links by removing the
`_templates/breadcrumbs.html` file: Sphinx and the RTD theme can
generate those links if the docs are correctly configured.
Add `"conf_py_path"` to the `html_context` dictionary in Sphinx
configuration file, so the links to the source files work correctly.
Minor updates on the configuration file to avoid some warnings: remove
the `language=None` and `html_theme_path`.
@santisoler
Copy link
Member Author

@lheagy this should fix those links without having to create our own Sphinx extension: RTD is capable of resolving those links out of the box if the configuration is correctly set. I noticed that the breadcrums.html file was overriding those links, using an invalid variable.

@santisoler santisoler merged commit d669b82 into main Dec 4, 2024
1 check failed
@santisoler santisoler deleted the fix-edit-on-github branch December 4, 2024 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant