Skip to content

Conversation

@vwheeler63
Copy link
Collaborator

@vwheeler63 vwheeler63 commented Sep 12, 2024

Links to Sublime Text and Visual Studio Code editors were not being displayed as intended because the link syntax was not being parsed by sphinx inside the boldfaced text block. Fix was to isolate links outside of boldfaced block. This was tested in an alternate sphinx build environment to confirm links now show up on page as intended.

An issue was not created about this.


📚 Documentation preview 📚: https://writethedocs-www--2217.org.readthedocs.build/

Links to `Sublime Text` and `Visual Studio Code` editors
were not being displayed as intended because the link
syntax was not being parsed by `sphinx` inside the boldfaced
text block.  Fix was to isolate links outside of boldfaced block.
This was tested in an alternate `sphinx` build environment
to confirm links now show up on page as intended.

An issue was not created about this.
@plaindocs plaindocs enabled auto-merge (squash) September 12, 2024 15:40
@plaindocs plaindocs merged commit f9d7a0f into writethedocs:main Sep 12, 2024
5 checks passed
@vwheeler63
Copy link
Collaborator Author

@plaindocs Hi! Quick question if you don't mind me asking:

What processes expressions like {% if theme_github_banner|lower != 'false' %} and variable substitution in {{ theme_github_user }} in docs/_templates/footer.html? Is it sphinx?

@vwheeler63 vwheeler63 deleted the fix/editor_links_in_contrib_doc branch September 12, 2024 16:20
@plaindocs
Copy link
Contributor

Yeah, those are part of the Sphinx theme, which is Alabaster. See https://github.com/writethedocs/www/blob/main/docs/conf.py#L118 and https://alabaster.readthedocs.io/en/latest/customization.html

@vwheeler63
Copy link
Collaborator Author

Yeah, those are part of the Sphinx theme, which is Alabaster.

Sweet! Thanks for letting me pick your brain. :-)

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.

2 participants