-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Readme sections in sidebar #535
Conversation
Personally, I do not think the redirect is needed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove javascript
@Blendify do you think the landing page needs some more content then? Without the redirect, readers will land on a page with only toctree that they need to click through. |
We can include the first couple of lines from the readme me? Or just
manually duplicate the content.
…On Jan 10, 2018 4:38 AM, "Jesse Tan" ***@***.***> wrote:
@Blendify <https://github.com/blendify> do you think the landing page
needs some more content then? Without the redirect, readers will land on a
page with only toctree that they need to click through.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#535 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AOeua8ikFSD_3HpGyy0SJI8SVdExrQ-sks5tJIT6gaJpZM4RPFVa>
.
|
Poke, I would just include an introductory statement in the index this should be adequate to describe the project. |
I'm waiting to see what the outcome of #532 is, since it may affect the way we can get the README to show up in the sidebar. |
The is obsolete now with 67bb54d |
This PR makes the README sections appear in the navbar.
It uses an intermediate file to indirectly include the README (
toctree
does not allow including files that are in a parent directory).The location of the README remains unchanged (so it renders nicely on GitHub).
A JavaScript redirect is used to send viewers directly to the README page, rather than displaying a page containing only the table of contents.
Fixes #523.