Skip to content

Conversation

@0marperez
Copy link
Contributor

Issue #

Description of changes

  • Graceful handling of empty list items in documentation trait

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@0marperez 0marperez added the no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly. label Nov 7, 2024
@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

@0marperez 0marperez marked this pull request as ready for review November 7, 2024 16:28
@0marperez 0marperez requested a review from a team as a code owner November 7, 2024 16:28
// If this list item holds a sublist, then we essentially just want to line break right away and
// render the nested list as normal.
val prefix = if (node.childNode(0).nodeName() == "ul") "\n" else ""
val prefix = if (childNode?.nodeName() == "ul") "\n" else ""
Copy link
Contributor

Choose a reason for hiding this comment

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

simplification: node.childNodes().firstOrNull()?.nodeName() == "ul"

@github-actions
Copy link

github-actions bot commented Nov 7, 2024

Affected Artifacts

No artifacts changed size

@0marperez 0marperez merged commit 3996a9b into main Nov 7, 2024
16 checks passed
@0marperez 0marperez deleted the documentation-empty-list-fix branch November 7, 2024 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants