Skip to content

Fixing links #989

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

Merged
merged 1 commit into from
Jan 13, 2018
Merged

Fixing links #989

merged 1 commit into from
Jan 13, 2018

Conversation

projektir
Copy link
Contributor

Fixing links for rust-lang/rust#46194.

@@ -9,7 +9,7 @@ Upon finding an `Err`, there are two valid actions to take:
1. `panic!` which we already decided to try to avoid if possible
2. `return` because an `Err` means it cannot be handled

`?` is *almost*[^1] exactly equivalent to an `unwrap` which `return`s
`?` is *almost*[^] exactly equivalent to an `unwrap` which `return`s
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So... this one is funky. There are two pages in RBE that are attempting to use the [^1] annotation. The way pulldown-cmark does annotations is by creating a div with id equal to the annotation name (in this case, "1"). This is all well and good, until the print.html page is created, in which case this now results in two different divs with id="1", which is invalid and is a failure to the linkchecker.

I've replaced 1 here with the dagger symbol , which looks like 1 in the fonts used by RBE, but I am not really sure how to address this issue in general since we basically can't use the 1, 2, 3 footnote style for as long as we intend to have a page that concatenates everything together.

@marioidival
Copy link
Member

LGTM, @QuietMisdreavus u can review and merge this?

Copy link
Member

@QuietMisdreavus QuietMisdreavus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. That's a real shame about the footnotes, though. We'll need to figure that one out eventually. :/

@Michael-F-Bryan
Copy link
Contributor

Looks good. That's a real shame about the footnotes, though. We'll need to figure that one out eventually. :/

It sounds like this isn't a RBE-specific issue, so we may be able to help out upstream. I've created an issue for it (rust-lang/mdBook#542).

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.

4 participants