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

Should the genindex be in the TOC? #59

Closed
hamishwillee opened this issue Apr 9, 2015 · 5 comments
Closed

Should the genindex be in the TOC? #59

hamishwillee opened this issue Apr 9, 2015 · 5 comments
Labels

Comments

@hamishwillee
Copy link
Contributor

This file is autogenerated, but not linked from anywhere http://python.dronekit.io/genindex.html

How useful is it? Should it be in the sidebar?

@mrpollo
Copy link
Member

mrpollo commented Apr 9, 2015

Probably below API reference? do you find it useful?

@hamishwillee
Copy link
Contributor Author

Not "super" useful - and this will be less useful when our class documentation CSS is improved and it is possible to read it more easily.

If we do want this in the TOC it has to be added as an explicit URL link to the index as shown below (ie you can't link to the version just built!):

Index <http://python.dronekit.io/genindex.html>

We could however just add it as a link on a home page or similar and then it can be linked "properly" using :ref:genindex.

I propose we do not add this. If neither of us are convinced it has clear value, then probably it is just clutter?

@mrpollo
Copy link
Member

mrpollo commented Apr 10, 2015

If you can add it as a submenu item of the api reference I vote to add it, if not lets leave it out

@hamishwillee
Copy link
Contributor Author

I can, included in pull request #58

Note, the included doc is hardcoded to point to the current python version of the file - so if you build this locally the link will take you to the main site. This is not desirable, but there is no way around it (as discussed above - you cannot include genindex in a TOC except by providing a full URL).

Closing, as this will be delivered by the pull request

@mrpollo
Copy link
Member

mrpollo commented Apr 13, 2015

thanks @hamishwillee

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants