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: line break logic #839

Closed
wants to merge 7 commits into from
Closed

Conversation

brandonkal
Copy link

@brandonkal brandonkal commented Sep 1, 2020

This PR fixes line break logic when formatting changes

Supplies requested previously failing test case for #751

The issue was that line breaks are inserted when switching on and off bold formatting.

It is not clear why this file is not formatted.
On its own to avoid noise.
This adds the failing tests for bold and linebreaks
@brandonkal brandonkal changed the title Failing tests fix: line break logic Sep 1, 2020
@brandonkal
Copy link
Author

@gitbrent updated PR with fix for the issue. This is good to merge.

@gitbrent
Copy link
Owner

gitbrent commented May 9, 2021

Can a test case for this issue be provided?

gnomical added a commit to Vigilant-LLC/PptxGenJS that referenced this pull request Jul 2, 2021
This is part of pull request gitbrent#839 in the source repo. applying in our fork until the source repo gets to managing its pull requests.
@gitbrent
Copy link
Owner

Closing due to age. Text/line-break algorithm has been completely rewritten in the past year.

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.

None yet

2 participants