nea / MarkdownViewerPlusPlus

A Notepad++ Plugin to view a Markdown file rendered on-the-fly
MIT License
1.17k stars 143 forks source link

More than 3 lines with bold text renders in the preview with extra return #98

Open reginvaldr opened 5 years ago

reginvaldr commented 5 years ago

Issue description

Type:

Data Centers are grouped into Availability Zones (backup & restore Availability Zones are grouped into Regions (e.g. Ireland)
High speed private links between Availability Zones
Data Centers are undisclosed
Edge locations are used for caching

Renders as: Data Centers are grouped into Availability Zones (backup & restore Availability Zones are grouped into Regions (e.g. Ireland) High speed private links between Availability Zones

Data Centers are undisclosed

Edge locations are used for caching

Markdown to reproduce

  Please post the markdown code producing the described issue, if any

Screenshot

If available

Environment

nea commented 5 years ago

Hey @reginvaldr

It is card to check this if not absolutely the same is parsed into this commen. Line breaks should only happen on double line breaks or double whitespaces at the end of a line.

Could you check that?

Thanks

reginvaldr commented 5 years ago

Yes, I understand. That is also what I expected. But after three lines text[spatie][spatie] it starts rendering the 4th line with an empty line (not just a line break)

So in Notepad type

text[space][space] text[space][space] text[space][space] text[space][space] text[space][space]

And you will see an empty line after 4th line

Op di 6 nov. 2018 om 16:15 schreef Savas Ziplies notifications@github.com

Hey @reginvaldr https://github.com/reginvaldr

It is card to check this if not absolutely the same is parsed into this commen. Line breaks should only happen on double line breaks or double whitespaces at the end of a line.

Could you check that?

Thanks

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nea/MarkdownViewerPlusPlus/issues/98#issuecomment-436288099, or mute the thread https://github.com/notifications/unsubscribe-auth/AZi4ID20sDEI4ICGhALu5ZgpJlMKxWIAks5usaeAgaJpZM4YQpM_ .

nea commented 5 years ago

Hi @reginvaldr

Ok, it seems the HTMLRenderer does that. The exported HTML looks correct. As I do not maintain that library it will be a tough one to fix, but I see now.

Thanks