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.

Sign up for GitHub

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

Jump to bottom

[gatsby-remark-autolink-headers] Headings with inline codes create links with redundant detail #5764

Closed
ryanditjia opened this issue Jun 6, 2018 · 1 comment
Closed

[gatsby-remark-autolink-headers] Headings with inline codes create links with redundant detail #5764

ryanditjia opened this issue Jun 6, 2018 · 1 comment
Labels
help wanted Issue with a clear description that the community can help with. type: bug An issue or pull request relating to a bug in Gatsby

Comments

@ryanditjia
Copy link
Contributor

ryanditjia commented Jun 6, 2018 •
edited
Loading

Description

Inline codes add unnecessary code-classlanguage-text code to its link. For example:

https://www.gatsbyjs.org/docs/migrating-from-v0-to-v1/#move-source-directories-pages-components-utils-etc-under-code-classlanguage-textsrccode

Should ideally be:

https://www.gatsbyjs.org/docs/migrating-from-v0-to-v1/#move-source-directories-pages-components-utils-etc-under-src
@ryanditjia ryanditjia mentioned this issue Jun 6, 2018
Merged
@m-allanson m-allanson added type: bug An issue or pull request relating to a bug in Gatsby help wanted Issue with a clear description that the community can help with. labels Jun 6, 2018
@pieh
Copy link
Contributor

pieh commented Jul 30, 2018 •
edited
Loading

This was fixed on gatsbyjs.org by #6856

I think it might be useful to add this to gatsby-remark-autolink-headers docs

---edit:
by "this" I mean moving gatsby-remark-autolink-headers before gatsby-remark-prismjs

@ryanditjia ryanditjia mentioned this issue Jul 30, 2018
Merged
piotrkwiecinski added a commit to codemanufacture/codemanufacture.com that referenced this issue Dec 18, 2018
@piotrkwiecinski
fix: move prismjs plugin after autolink headers in gatsby config
65d8301 
To make sure we don't have issue with redundant details.
See: gatsbyjs/gatsby#5764
@Ameobea Ameobea mentioned this issue Apr 24, 2019
Closed
morenyang added a commit to morenyang/blog that referenced this issue Oct 3, 2019
@morenyang
Fix auto-link url issue
e2b1560 
refs: gatsbyjs/gatsby#5764
@vihanb vihanb mentioned this issue Jun 4, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Issue with a clear description that the community can help with. type: bug An issue or pull request relating to a bug in Gatsby
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@m-allanson @pieh @ryanditjia