Styled links are all broken in Web IDL source docs
Categories
(Core :: DOM: Bindings (WebIDL), defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox95 | --- | fixed |
People
(Reporter: saschanaz, Assigned: saschanaz)
References
Details
Attachments
(5 files, 2 obsolete files)
https://firefox-source-docs.mozilla.org/dom/webIdlBindings/
Texts are rendered like:
All WebIDL files should be placed in `dom/webidl https://searchfox.org/mozilla-central/source/dom/webidl/`__ and added to the list in the moz.build file in that directory.
Which is just wrong.
I'm not sure how to fix that in reStructuredText format, and I'm not sure what value is there to keep that format when Markdown support is already there in source docs.
How about converting it to Markdown?
Assignee | ||
Updated•3 years ago
|
Comment 1•3 years ago
|
||
Per https://www.sphinx-doc.org/en/master/usage/restructuredtext/basics.html#gotchas, reStructuredText doesn't support nested inline markup. It seems like it doesn't support a mix of inline markup and hyperlink, either.
I have no opinion on converting it to Markdown. What do you think, :peterv?
Comment 2•3 years ago
|
||
Yeah, reStructuredText doesn't have an easy way to support that as far as I could find (see https://docutils.sourceforge.io/FAQ.html#is-nested-inline-markup-possible). I didn't remove the broken formatting, because the original formatting had some value and I was hoping a solution would turn up eventually.
I'm not opposed to converting to Markdown, although I think all the other documentation in dom already uses reStructuredText. So it seems like the value that Markdown would bring isn't exactly overwhelming either.
Assignee | ||
Comment 3•3 years ago
|
||
First converted by pandoc index.rst -o index.md -t gfm
, formatted by Prettier with --prose-wrap=always
, and then did some manual fix for the broken links and the C++ reflection table.
Updated•3 years ago
|
Comment 4•3 years ago
|
||
First converted with pandoc index.rst -o index.md -t gfm
, did some manual fix for the broken links and the C++ reflection table, and then formatted with Prettier by --prose-wrap=always
.
Assignee | ||
Comment 5•3 years ago
|
||
(In reply to Peter Van der Beken [:peterv] from comment #4)
Created attachment 9238299 [details]
WIP: Bug 1726440 - Convert IDL Bindings docs to MarkdownFirst converted with
pandoc index.rst -o index.md -t gfm
, did some manual fix for the broken links and the C++ reflection table, and then formatted with Prettier by--prose-wrap=always
.
Thank you for doing that for me!
Comment 6•3 years ago
|
||
Assignee | ||
Comment 7•3 years ago
|
||
Sorry, I don't see which part of it uses styled links, could you point the line number? (specifically this kind of style
.)
Comment 9•3 years ago
|
||
Kagami, it seems this might have gotten lost, but also seems worthwhile to finish?
Assignee | ||
Comment 10•3 years ago
•
|
||
I'll try splitting the patch since it includes something more than conversion e.g. assigning proper language for code blocks and removing redundant section ids.
And probably applying prettier also can be a separate step.
Assignee | ||
Comment 12•3 years ago
|
||
Those ids were generated by MDN but nothing really uses them.
Depends on D127656
Assignee | ||
Comment 13•3 years ago
|
||
Depends on D127688
Assignee | ||
Comment 14•3 years ago
|
||
Depends on D127689
Assignee | ||
Comment 15•3 years ago
|
||
Depends on D127690
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Assignee | ||
Comment 16•3 years ago
|
||
Updated•3 years ago
|
Comment 17•3 years ago
|
||
Pushed by krosylight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/41c92a7c3667 Part 1: Remove unused rst references r=peterv DONTBUILD https://hg.mozilla.org/integration/autoland/rev/3bf9463f2fa4 Part 2: Assign proper languages to code directives r=peterv DONTBUILD https://hg.mozilla.org/integration/autoland/rev/140a93caa8a3 Part 3: Fix some typo r=peterv DONTBUILD https://hg.mozilla.org/integration/autoland/rev/e379fb6e91d2 Part 4-1: Rename from .rst to .md r=peterv DONTBUILD https://hg.mozilla.org/integration/autoland/rev/e8d7db0d2ca4 Part 4-2: Migrate to Markdown r=peterv DONTBUILD
Comment 18•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/41c92a7c3667
https://hg.mozilla.org/mozilla-central/rev/3bf9463f2fa4
https://hg.mozilla.org/mozilla-central/rev/140a93caa8a3
https://hg.mozilla.org/mozilla-central/rev/e379fb6e91d2
https://hg.mozilla.org/mozilla-central/rev/e8d7db0d2ca4
Description
•