Skip to content

Remove 'navigation'#258

Closed
methane wants to merge 1 commit into
python:mainfrom
methane:remove-navigation
Closed

Remove 'navigation'#258
methane wants to merge 1 commit into
python:mainfrom
methane:remove-navigation

Conversation

@methane

@methane methane commented Sep 8, 2025

Copy link
Copy Markdown
Member

I found this in Transifex. This is not displayed. (e.g. display: none).


📚 Documentation preview 📚: https://python-docs-theme-previews--258.org.readthedocs.build/

@encukou

encukou commented Sep 8, 2025

Copy link
Copy Markdown
Member

It might be there for screen readers.

@methane

methane commented Sep 8, 2025

Copy link
Copy Markdown
Member Author

Generally, screen readers ignores display: none.
div.related h3 { display: none } lives since first commit of Sphinx.
sphinx-doc/sphinx@d60ca8d#diff-9b474a3f571175e217231911be3927a1e66e79b84b4a98c95a4788b475c604bdR68

@methane

methane commented Sep 8, 2025

Copy link
Copy Markdown
Member Author

Perhaps they're using h3 tags to indicate that this section is the place for navigation links when the css isn't loaded.
If so, it would be best to translate this part as well, just like the displayed text.

@methane methane closed this Sep 8, 2025
@methane methane deleted the remove-navigation branch September 8, 2025 15:23
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