Skip to content

platform: PT-3799 Remove extra space before inline markers and annotations#454

Merged
jolierabideau merged 6 commits intomainfrom
pt-3799-extra-space-before-markers
Feb 27, 2026
Merged

platform: PT-3799 Remove extra space before inline markers and annotations#454
jolierabideau merged 6 commits intomainfrom
pt-3799-extra-space-before-markers

Conversation

@jolierabideau
Copy link
Copy Markdown
Contributor

@jolierabideau jolierabideau commented Feb 25, 2026

  • Make TextSpacingPlugin skip adding a trailing space when the TextNode's next sibling is a CharNode or TypedMarkNode
  • Update $textNodeTrailingSpaceTransform function description
  • Add tests for: preserving space-only TextNodes when next sibling is a CharNode, not adding trailing space when next sibling is a CharNode, not adding trailing space when next sibling is a TypedMarkNode
  • Fix typo useUsfmMakersForMenu to useUsfmMarkersForMenu in shared-react library and scribe package.

Open with Devin

@codesandbox
Copy link
Copy Markdown

codesandbox bot commented Feb 25, 2026

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@jolierabideau jolierabideau changed the title PT-3799 Remove extra space before inline markers and annotations platform: PT-3799 Remove extra space before inline markers and annotations Feb 25, 2026
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

Copy link
Copy Markdown
Collaborator

@irahopkinson irahopkinson left a comment

Choose a reason for hiding this comment

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

Good job. Please consider other inline nodes as per my comment.

@irahopkinson
Copy link
Copy Markdown
Collaborator

Also if you are going to do a Squash and merge then we are fine but if you are going to Create a merge commit then we need to do a little git magic first. Let me know if you need help with that.

Copy link
Copy Markdown
Collaborator

@irahopkinson irahopkinson left a comment

Choose a reason for hiding this comment

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

👍 LGTM thanks for this.

@jolierabideau jolierabideau merged commit 9f09780 into main Feb 27, 2026
5 checks passed
@jolierabideau jolierabideau deleted the pt-3799-extra-space-before-markers branch February 27, 2026 02:07
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.

2 participants