associate BlockMarkup.md with the BlockMarkup type#158
associate BlockMarkup.md with the BlockMarkup type#158tayloraswift wants to merge 5 commits intoswiftlang:mainfrom
Conversation
d-ronnqvist
left a comment
There was a problem hiding this comment.
Since articles are phased as a list of all the nodes I wonder if it's be more according to the original intent to just rename those files to "block-markup-nodes.md" and "inline-markup-nodes.md".
Also, @DocumentationExtension(mergeBehavior: append) is the default and is never needed. Please remove that.
|
i don’t think these these stubs would be very useful as articles on their own, so i’m assuming the intent was just to add some custom curation to these types. i’ve removed the redundant block directives. |
|
I think the intent was for these pages to be "API collections" just like the "Infrastructure", "Visiting Markup", and "Formatter and Options" pages. An API collection is authored in a markdown file, just like an article, but its purpose is to organize related symbols rather than to describe how to do something. You can find more examples of API collections in Apple's documentation, for example here in UIKit and here in SwiftUI. |
|
cc @Kyle-Ye |
Here is what I got from @tayloraswift 'description. The name of our article page is the same as some protocol in Markdown framework which will cause some strange rending issue when conforming the protocol elsewhere. |
Kyle-Ye
left a comment
There was a problem hiding this comment.
The initial intend for these file is to serve as an article. We reference them in Markdown.md with the first one instead of the second.
- <doc:BlockMarkup>
- ``BlockMarkup``
So I'd like to suggest you update the filename of article and update those articles reference in Markdown.md
eg. InlineMarkup.md -> Markup Inline Nodes.md and <doc:Markup-Inline-Nodes>
this markdown article is running into a DocC bug where it replaces the
BlockMarkupsymbol, which causes some rather strange rendering:this PR associates the article properly to the
BlockMarkupsymbol by turning the article into a markdown supplement.