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

Search result page must not contain "Edit on GitHub" Link. #934

Closed
dojutsu-user opened this issue Mar 4, 2019 · 3 comments · Fixed by #935
Closed

Search result page must not contain "Edit on GitHub" Link. #934

dojutsu-user opened this issue Mar 4, 2019 · 3 comments · Fixed by #935

Comments

@dojutsu-user
Copy link
Member

"Edit on GitHub" link (top right corner) must not be present on Search results page. Also it leads to 404 pages.

screenshot from 2019-03-04 16-06-22

@Mike-Dai
Copy link

Mike-Dai commented Mar 5, 2019

Does anyone work on this issue? If not, I'd like to take this issue.

@rshrc
Copy link

rshrc commented Mar 5, 2019

@stsewd https://github.com/rshrc/sphinx_rtd_theme/blob/master/sphinx_rtd_theme/breadcrumbs.html
maybe, it is here, I think.
{% if check_meta and 'github_url' in meta %} {% set display_github = True %} {% endif %}

@stsewd stsewd transferred this issue from readthedocs/readthedocs.org Jun 1, 2020
stsewd added a commit that referenced this issue Jun 1, 2020
This page is autogenerated, so it doesn't have "a real page".

Fixes #934
eine pushed a commit to buildthedocs/sphinx.theme that referenced this issue Jul 10, 2020
This page is autogenerated, so it doesn't have "a real page".

Fixes readthedocs#934
SimonMellerin pushed a commit to makinacorpus/makina-sphinx-theme that referenced this issue Nov 19, 2020
This page is autogenerated, so it doesn't have "a real page".

Fixes readthedocs/sphinx_rtd_theme#934
Donald-Ennis pushed a commit to Donald-Ennis/Sphinx-Theme that referenced this issue Feb 8, 2025
This page is autogenerated, so it doesn't have "a real page".

Fixes readthedocs/sphinx_rtd_theme#934
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 a pull request may close this issue.

4 participants