Skip to content

Support for markdown function list - #405

Closed
pcpro178 wants to merge 5 commits into
notepad-plus-plus:masterfrom
pcpro178:jfell/markdown-functionlist
Closed

Support for markdown function list#405
pcpro178 wants to merge 5 commits into
notepad-plus-plus:masterfrom
pcpro178:jfell/markdown-functionlist

Conversation

@pcpro178

Copy link
Copy Markdown

This pull request adds support for markdown (.md) formatted file to the Notepad++ Function List. Markdown heading rows are displayed in the Function List, as shown in the following image.

image

@pcpro178
pcpro178 marked this pull request as ready for review August 28, 2026 02:39
@pryrt

pryrt commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Remove overrideMap.xml; it should not get added to the repo.

Must update the JSON

@pryrt pryrt added the needs improvement will be rejected if requested improvement isn't made label Aug 28, 2026
@pcpro178

Copy link
Copy Markdown
Author

@pryrt I removed the override, but I'm not sure how the function list will work now, since Markdown is a preinstalled language. I updated the JSON.

@pryrt

pryrt commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

It's complaining about the --- in the underline comment in the function list XML file, not the single hyphen in the JSON. <!-- blah --- blah --> is not valid xml the first two hyphens of the triple technically end the comment, and everything that follows is invalid xml

https://stackoverflow.com/a/10842176

@pcpro178 pcpro178 closed this Aug 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs improvement will be rejected if requested improvement isn't made

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants