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

Navigation (+other) improvements to Documentation #4855

Merged
merged 6 commits into from
Nov 12, 2017

Conversation

pradyunsg
Copy link
Member

Closes #4852

@pradyunsg pradyunsg added the type: docs Documentation related label Nov 11, 2017
@pradyunsg
Copy link
Member Author

pradyunsg commented Nov 11, 2017

I had added the Contents tree and removed it because... well, see for yourself:

screen shot 2017-11-11 at 10 51 59 am

@pradyunsg pradyunsg added the skip news Does not need a NEWS file entry (eg: trivial changes) label Nov 11, 2017
@pradyunsg
Copy link
Member Author

pradyunsg commented Nov 11, 2017

Uhm... why is everything so fragile.

#4755 "broke" this build because it sees that pip's documentation build is called with sphinx-build as the executable. None of the CI tools would detect this; it's visible as sphinx-build install in the above image.

I'm gonna put pip in place of the get_prog call because, well, it isn't supposed to return anything else here anyway.

@pfmoore
Copy link
Member

pfmoore commented Nov 11, 2017

Not sure what's wrong with the contents tree, apart from it should go after the header and the styling of the word "Contents" should match the other headings. Also, the sidebar only has one extra level - I'd strongly prefer to have full navigation from the sidebar. (But ideally not all fully expanded - does the theme support that?)

@pradyunsg
Copy link
Member Author

(But ideally not all fully expanded - does the theme support that?)

Nope. Putting any more levels in fact made it look weird to me.

Not sure what's wrong with the contents tree

How it looks -- the styling. I guess, I should get over that.

apart from it should go after the header

Will do.

the styling of the word "Contents" should match the other headings.

Would need to happen in the theme.

@pradyunsg
Copy link
Member Author

Deployed this branch at http://pip-pradyunsg.readthedocs.io

@pradyunsg
Copy link
Member Author

I'd strongly prefer to have full navigation from the sidebar.

clarifying that when enabled, this looks weird to me -- the sidebar content wrap along multiple lines.

Copy link
Member

@pfmoore pfmoore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks OK to me. I would still prefer Javascript-based dynamic expansion in the sidebar, like the old theme had, but that's a problem with the new theme, and not one that I have the time to pursue, so I'm OK with accepting it as it is.

There is one check that needs fixing first, of course :-)

@pradyunsg
Copy link
Member Author

Thanks @pfmoore! That should fix it.

I defer to you for clicking merge on this. :)

@pfmoore pfmoore merged commit 8f6b4c9 into pypa:master Nov 12, 2017
@pradyunsg pradyunsg deleted the docs/navigation-improvements branch November 12, 2017 19:41
@pradyunsg
Copy link
Member Author

If anyone has the time to pursue, I'd say they should fix it over at docs.python.org.

@pradyunsg pradyunsg restored the docs/navigation-improvements branch May 27, 2018 09:38
@pradyunsg pradyunsg deleted the docs/navigation-improvements branch June 17, 2018 06:48
@lock
Copy link

lock bot commented Jun 2, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Jun 2, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jun 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation skip news Does not need a NEWS file entry (eg: trivial changes) type: docs Documentation related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New docs theme doesn't show subsections under (for example) pip install
2 participants