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
docs: Use inline image link in faq.md #35413
Conversation
💖 Thanks for opening this pull request! 💖 We use semantic commit messages to streamline the release process. Before your pull request can be merged, you should update your pull request title to start with a semantic prefix. Examples of commit messages with semantic prefixes:
Things that will help get your PR across the finish line:
We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can. |
@apetresc could you please push an empty commit to trigger the lint check? |
Done! (The Appveyor build failures seem totally unrelated to the change 🤔) |
@apetresc Can you rebase this PR on latest |
Thanks for doing the rebase for me - I'm travelling at the moment without my laptop until tomorrow. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this is the right approach to fixing this broken image link. All image links in the docs are relative links, they're transformed by Docusaurus (I believe, @erickzhao correct me if I'm wrong) to point to the correct asset paths (which include hashes). Changing this to an absolute link sidesteps that process.
My guess would be Docusaurus isn't transforming the image path correctly due to this case using a reference-style link, or because it's using the simplified syntax style (![]
rather than ![][]
) - there might be a bug in Docusaurus, or our own code which fixes up links.
I think changing this to an inline link would fix the issue and ensure the image goes through the same asset processing as the other images in the docs.
That makes sense, thanks @dsanders11. I'm not sure what the underlying Docusaurus problem is that causes the reference-style link to fail to be transformed properly, but if switching to an inline link works, that's good (and it seems like that's the likely culprit since other relative inline image links in other tutorial pages render fine in the HTML site). So, I've updated this patch to do just that 🙂 |
Should I try rebasing again to resolve the appveyor build failures? |
@apetresc, you can rebase or push an empty commit. |
@dsanders11 Looks like appveyor finally passed now but the Semver job is hanging because the PR somehow ended up with multiple semver labels. Is that something you have permissions to fix? |
The relative link is rendered relative to the host domain, which works fine when viewing it on Github, but since you also use the same generated HTML in your doc site, the link is broken. See here: https://www.electronjs.org/docs/latest/faq#the-font-looks-blurry-what-is-this-and-what-can-i-do Using an absolute URL here should fix the issue on the main site.
Yaaay it finally passed all the checks 🎊 🥳 🎊 |
Congrats on merging your first pull request! 🎉🎉🎉 |
Release Notes Persisted
|
I have automatically backported this PR to "19-x-y", please check out #35647 |
I have automatically backported this PR to "21-x-y", please check out #35648 |
I have automatically backported this PR to "20-x-y", please check out #35649 |
* Use absolute URL in faq.md image link The relative link is rendered relative to the host domain, which works fine when viewing it on Github, but since you also use the same generated HTML in your doc site, the link is broken. See here: https://www.electronjs.org/docs/latest/faq#the-font-looks-blurry-what-is-this-and-what-can-i-do Using an absolute URL here should fix the issue on the main site. * Use inline image reference for subpixel rendering example As suggested by @dsanders11
Description of Change
Relative Markdown links are rendered relative to the host domain, which works fine when viewing it on Github, but since you also use the same generated HTML in your doc site, the link is broken. See here: https://www.electronjs.org/docs/latest/faq#the-font-looks-blurry-what-is-this-and-what-can-i-do
Using an absolute URL here should fix the issue on the main site.
Checklist
npm test
passesRelease Notes
Notes: Fixed broken image link in
docs/latest/faq
.