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

rustdoc: Fix broken link to FontFace in components/gfx #31745

Merged
merged 3 commits into from Mar 19, 2024

Conversation

oluwatobiss
Copy link
Contributor

@oluwatobiss oluwatobiss commented Mar 18, 2024

Add the MDN documentation link referenced in the rustwasm doc to resolve the FontFace broken link warning.


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes are part of Fix as many rustdoc errors as possible #31575 (GitHub issue number if applicable)
  • These changes do not require tests because they are documentation fixes.

Comment on lines 84 to 86
///
/// [`FontFace`]: https://developer.mozilla.org/en-US/docs/Web/API/FontFace
///
Copy link
Member

Choose a reason for hiding this comment

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

This isn't the FontFace that's referred to here. You can just use dwrote::FontFace, I think.

Copy link
Member

Choose a reason for hiding this comment

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

Actually, the issue here is that the method is returning Font, but I had written FontFace when I added this documentation because it changed while refactoring.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it. Thanks for letting me know.

@mrobinson mrobinson changed the title rustdoc: Fix fontface broken link error in components/gfx rustdoc: Fix broken link to FontFace in components/gfx Mar 19, 2024
@mrobinson mrobinson added this pull request to the merge queue Mar 19, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 19, 2024
@mrobinson mrobinson added this pull request to the merge queue Mar 19, 2024
Merged via the queue into servo:main with commit d3b03a2 Mar 19, 2024
9 checks passed
@oluwatobiss oluwatobiss deleted the rustdoc/fix-fontface-error branch March 20, 2024 00:57
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.

None yet

2 participants