Add option to remove linebreaks within pagragraph tags #186
Conversation
| shouldn't</span>` | ||
|
|
||
| expected := `<p>This newline should be ignored</p> | ||
| <span>And this one |
There was a problem hiding this comment.
Is this actually what is expected to be a result? Should not it be like this newline\tshould be ignored? As far as I see, the main idea was to remove only linebreaks.
There was a problem hiding this comment.
As far as I'm aware, the goal to to avoid weird breaks in the confluence behavior; that involves flowing text in <p>, so \t should be removed as well
There was a problem hiding this comment.
We'd like this feature as well but I agree that it should only replace linebreaks and not all space. The regex is incorrect.
What's the status of this? It hasn't had updates in a while so looks stale. Wasn't sure if I should put up another PR
|
I'd really like to see this feature land. @GusAntoniassi are you still working on this? |
This merge request addresses #92 by adding the flag
--drop-single-linebreakwhich when enabled will strip all\ncharacters within<p></p>tags.Suggestions or changes to the flag name are welcome.