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

[docs] Add "Edit on GH" links, fix searchbox, widen viewport, better legibility #833

Merged
merged 8 commits into from
Mar 28, 2022

Conversation

ix5
Copy link
Member

@ix5 ix5 commented Mar 28, 2022

For a visual comparison see:

Before:
Screen Shot 2022-03-28 at 21 47 21-fullpage

After:
Screen Shot 2022-03-28 at 21 47 03-fullpage

@ix5 ix5 added feature docs Documentation of the project labels Mar 28, 2022
@ix5 ix5 added this to the 0.13 milestone Mar 28, 2022
ix5 added 8 commits March 28, 2022 21:52
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
This also necessitates a bit of CSS.
This increases legibility and improves contrast.
The implementation closely matches `sphinx-rtd-theme`,
see https://github.com/readthedocs/sphinx_rtd_theme/blob/9264091087620d421b0804c00937b00980ac3916/sphinx_rtd_theme/breadcrumbs.html#L19-L40
and https://github.com/readthedocs/sphinx_rtd_theme/blob/ef9a0584ef6d43f898a2b787a541e31a79f560e8/docs/configuring.rst#file-wide-metadata

GitHub svg icon taken from Wikipedia (Wikimedia Commons)

Usage: https://github.com/logos
> Do these awesome things:
> - Use the Octocat or GitHub logo to link to GitHub
> - Use the Mark in social buttons to link to your GitHub
>   profile or project
> - Use the Octocat or GitHub logo to advertise that your
>   product has built-in GitHub integration
> - Use the Octocat or GitHub logo in a blog post or news
>   article about GitHub
A name that's broken across two lines looks ugly
@ix5 ix5 merged commit d2381b3 into isso-comments:master Mar 28, 2022
@ix5 ix5 deleted the docs-theme-improve branch March 28, 2022 19:53
@ix5 ix5 mentioned this pull request Apr 30, 2022
38 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation of the project feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant