Skip to content

parsedown extension#318

Closed
dav-m85 wants to merge 1 commit intopicocms:masterfrom
dav-m85:feature/parsedown-extension
Closed

parsedown extension#318
dav-m85 wants to merge 1 commit intopicocms:masterfrom
dav-m85:feature/parsedown-extension

Conversation

@dav-m85
Copy link
Contributor

@dav-m85 dav-m85 commented Jan 23, 2016

I needed a way of extending the Parsedown extension, hence this PR. Not sure if it is the proper way though.

@PhrozenByte
Copy link
Collaborator

A plugin developer shouldn't be able to replace the Markdown parser with something different (what would be possible with a Pico::setParsedown() method). Unfortunately I don't have a ironclad solution either. You can find my current ideas in #317 (3rd item of "Event system"). I really appreciate your feedback about those ideas!

@PhrozenByte
Copy link
Collaborator

We'll implement this feature by adding a new onParsedownRegistered(Parsedown $parsedown) event to Pico 1.1. Please refer to #334 for further discussions about this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants