-
Notifications
You must be signed in to change notification settings - Fork 143
Add row and column directives #378
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
Conversation
Articles currently don’t support having a directive as the first item of the discussion. This resolves the issue while maintaining compatibility with Metadata, Comment, and DeprecationSummary blocks.
94729dd
to
584b38b
Compare
@swift-ci please test |
584b38b
to
29531ae
Compare
@swift-ci please test |
Adds support for the @Row and @column directive as described here: https://forums.swift.org/t/supporting-more-dynamic-content-in-swift-docc-reference-documentation/59527#row-9. Resolves rdar://97739637
29531ae
to
52e9fed
Compare
@swift-ci please test |
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.
Looks great! Just a few suggestions
@@ -60,6 +60,8 @@ public enum RenderBlockContent: Equatable { | |||
case termList(TermList) | |||
/// A table that contains a list of row data. | |||
case table(Table) | |||
|
|||
case row(Row) |
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.
Looks like we're missing docs here, same for the public APIs below
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.
Thanks! Addressed here: d0c2e31.
@swift-ci please test |
Bug/issue #, if applicable: rdar://97739637
Summary
Adds support for
@Row
and@Column
in reference documentation pages.Described on the forums here: https://forums.swift.org/t/supporting-more-dynamic-content-in-swift-docc-reference-documentation/59527#row-9.
Dependencies
@Row
directive swift-docc-render#409