MarkdownViewerPlusPlus icon indicating copy to clipboard operation
MarkdownViewerPlusPlus copied to clipboard

Bug report: spacing between list items (unordered or ordered) breaks the Markdown rendering

Open xmatic opened this issue 5 years ago • 4 comments

Issue description

When creating lists (unordered or ordered), if one adds line spacing between lines, it breaks the Markdown rendering.

This is best explained via screenshots, so I'm including screenshots:

This works fine: https://imgur.com/EcYgjbh

This is broken: https://imgur.com/hMwJTPU

Screenshot

This is fine: https://imgur.com/EcYgjbh

This is broken: https://imgur.com/hMwJTPU

Environment

  • Notepad++ 7.5.9 (x64)_
  • MarkdownViewer++ 0.8.2.25564
  • Windows 10 (x64)_

xmatic avatar Nov 13 '20 20:11 xmatic

I have the exact same problem Notepad++ 7.8.4 (x64) Markdown viewer 0.8.2.25525 The HTML simply has

tags in the
  • item, which is allowed. So I guess it's just an HTML rendering issue? Is it worth trying the pull request version I see has an enhanced renderer. I have not found time yet.
  • mullocksj avatar Nov 19 '20 16:11 mullocksj

    Oops! should have used preview I meant to write

    simply has <p> </p> tags in the <li> tag which is allowed.

    mullocksj avatar Nov 19 '20 16:11 mullocksj

    @mullocksj I would certainly give it a try. But I've never merged a pull-request myself. Not sure how that works. Would one fork the original project and then merge the pull request in the fork?

    xmatic avatar Dec 05 '20 02:12 xmatic

    @xmatic Sorry I am a Github newbie. I did find how to download the source via the web interface. "Pull requests" link shows you the single pull request. Click on that to see more detail and a line near the top reads "charliedrewitt wants to merge 11 commits into nea:master from charliedrewitt:master". The "charliedrewitt:master" part is a link to a view of that branch, where there is a download button.

    Date sent: Fri, 04 Dec 2020 18:46:20 -0800 From: xmatic [email protected] Send reply to: nea/MarkdownViewerPlusPlus [email protected] To: nea/MarkdownViewerPlusPlus [email protected] Copies to: mullocksj [email protected], Mention [email protected] Subject: Re: [nea/MarkdownViewerPlusPlus] Bug report: spacing between list items (unordered or ordered) breaks the Markdown rendering (#132)

    @mullocksj I would certainly give it a try. But I've never merged a pull-request myself. Not sure how that works. Would one fork the original project and then merge the pull request in the fork?

    You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

    -- Dr. S.J. Mullock http://www.kore.co.uk Kore Technology Ltd Cambridgeshire Business Park Ely, Cambs. CB7 4EA, UK Tel. (Int.+ 44) (0) 1353 653030 Fax. (Int.+ 44) (0) 1353 653031 Registered in England, No. 2643175

    mullocksj avatar Dec 07 '20 09:12 mullocksj