Skip to content

Conversation

@sbesson
Copy link
Member

@sbesson sbesson commented Nov 20, 2012

Add two links below the global table of contents.

@melissalinkert
Copy link
Member

No objections from me.

@joshmoore
Copy link
Member

@hflynn, any of other changes for this PR or do you want to handle that after more discussions, etc?

@hflynn
Copy link
Contributor

hflynn commented Nov 21, 2012

Ok to merge

@hflynn
Copy link
Contributor

hflynn commented Nov 21, 2012

Scratch that - links need reformatting

@sbesson
Copy link
Member Author

sbesson commented Nov 22, 2012

@hflynn , @qidane : Note the commit 5b72cde above solves the breadcrumps/navigation bar duplication issue but removes any kind of HTML navigation (next/previous).

@manics
Copy link
Member

manics commented Nov 22, 2012

Looks OK to me. My personal preference is to remove the "Show Source" links since it doesn't seem that useful to users, and put Prev/Next links somewhere in the side bar. I'll let you decide... if you're happy with it this way then ready to merge.

@hflynn
Copy link
Contributor

hflynn commented Nov 22, 2012

@sbesson is going to add Prev/Next to the side bar. 'Show source' is useful for debugging apparently so that is going to stay for now.

@manics
Copy link
Member

manics commented Nov 22, 2012

Ready to merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants