Skip to content
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

Fix source span calculation for LineBreakInline #760

Merged

Conversation

zickb
Copy link

@zickb zickb commented Dec 14, 2023

Fix the source span calculation of the last LineBreakInline in paragraph when multiple blocks exists.

I have added the source span calculation for the last LineBreakInline in #733, but the logic does not handle multiple blocks correctly. This corrects the logic.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 7203562141

  • 28 of 28 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.004%) to 93.146%

Totals Coverage Status
Change from base Build 7185576836: 0.004%
Covered Lines: 26047
Relevant Lines: 27331

💛 - Coveralls

@xoofx xoofx added the bug label Dec 14, 2023
@xoofx xoofx merged commit a092ec2 into xoofx:master Dec 14, 2023
1 check passed
@xoofx
Copy link
Owner

xoofx commented Dec 14, 2023

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants