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

Remove HTML P at end of generated HTML #6

Open
zipwiz opened this issue Apr 14, 2020 · 1 comment
Open

Remove HTML P at end of generated HTML #6

zipwiz opened this issue Apr 14, 2020 · 1 comment

Comments

@zipwiz
Copy link
Contributor

zipwiz commented Apr 14, 2020

Actually the generated HTML is finished by a single "<p>".

The technical reason for this is that the implemantation simply adds an opening "<p>" just in case there is still some text to come.

Syntactically spoken this is dangerous because there may follow other HTML running into problems with this "opened but not closed <p>". This finishing "<p>" should be removed therefore as soon as it is sure that the RTF source is completely consumed. This may be done at the very end of the transformation, just befor returning the result.

@zipwiz
Copy link
Contributor Author

zipwiz commented Apr 14, 2020

(I do not think that I will be able to work on this the next time because nearly all tests have to be adjusted.)

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

No branches or pull requests

1 participant