Skip to content

Conversation

curtismitchell
Copy link
Contributor

  • Fixes the bug where the usesidemenu config option should be considered before creating jump links
  • Moves the links to before the headings
  • Limits the application of the links to H2 headings (for now)

@Dynalon
Copy link
Owner

Dynalon commented Dec 6, 2013

The problem when putting the jump link before the heading is that on pages that have a 2nd degree heading at the top, will also have jump link which does not make much sense. An example is the layout.md page in the current MDwiki Wiki:

@Dynalon
Copy link
Owner

Dynalon commented Dec 7, 2013

I've thrown out the 2 commits that did nothing because reverted themself, and squashed merged the other to fixes into single commit 5ba2bc5

Thanks for the fixes!

P.S: You might want to enable trailing whitespace highlighting in your editor, as there were empty lines with whitespace in your pull request ;)

@Dynalon Dynalon closed this Dec 7, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants