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

Using TOC in article breaks link in home page #23

Closed
Marrin opened this issue Sep 11, 2013 · 6 comments
Closed

Using TOC in article breaks link in home page #23

Marrin opened this issue Sep 11, 2013 · 6 comments

Comments

@Marrin
Copy link

@Marrin Marrin commented Sep 11, 2013

If a .. contents:: directive is used within an article the headline on the blog home page does not link to the article page any more.

@christianjann
Copy link
Contributor

@christianjann christianjann commented Sep 11, 2013

Could you give an example (upload it somewhere or attach a html file), I'm using the .. contents:: directive in nearly every blog post and I can't understand the problem at the moment.

@Marrin
Copy link
Author

@Marrin Marrin commented Sep 11, 2013

The second blog post on http://blog.marc.rintsch.de shows the symptom. It is not using the very latest tinkerer version yet but the problem is the same.

@christianjann
Copy link
Contributor

@christianjann christianjann commented Sep 11, 2013

It should work if you replace

.. contents:: Inhalt

with

.. contents:: Inhalt
  :depth: 2
  :local:

and it also looks a bit better (my opinion). This is no real fix, but let me know whether this fixes your problem.

PS: I would add a bit CSS from Sphinx to style the contents directive a bit, you can look for example into the dark theme:

div.topic {
    border: 1px solid #ccc;
    padding: 7px 7px 0 7px;
    margin: 10px 0 10px 0;
}

div.topic {
    background: none repeat scroll 0 0 #F8F8F8;
}

p.admonition-title, p.topic-title {
    font-size: 1.1em;
    font-weight: bold;
    margin: 0px 10px 5px 0px;
}

@Marrin
Copy link
Author

@Marrin Marrin commented Sep 11, 2013

Yes, that fixes the link problem. Thanks.

@christianjann
Copy link
Contributor

@christianjann christianjann commented Sep 12, 2013

Nice. Hey @vladris I can't close the issue so if you're OK with that, then you can close it.

@vladris
Copy link
Owner

@vladris vladris commented Sep 12, 2013

Sure. Thanks for your prompt reply Christian!

@vladris vladris closed this Sep 12, 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
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants