Skip to content

Conversation

alansouzati
Copy link
Contributor

Hi guys,

Great job in the markdown loader. I'm trying to use more options from marked module: highlight attribute which expects a function (similar to the renderer). I was unable to send such a function as a query param.

This PR aims to extend the functionality already in place for renderer function. With my changes, all regular marked options are accepted inside markdownLoader object.

Alan

@valorkin
Copy link

valorkin commented Sep 3, 2015

@jhnns please review, merge and publish this PR
using custom highlight is pretty hard without it

CC: @matthaias @meaku @sbat @topa or some one from this team:)

@meaku
Copy link
Member

meaku commented Sep 4, 2015

Looks good! Thank you!

meaku added a commit that referenced this pull request Sep 4, 2015
Added support for full set of marked options.
@meaku meaku merged commit 5a82a16 into peerigon:master Sep 4, 2015
@valorkin
Copy link

valorkin commented Sep 4, 2015

@meaku will you be able to publish this changes? please :)

@meaku
Copy link
Member

meaku commented Sep 4, 2015

Released as 0.1.7.

@valorkin
Copy link

valorkin commented Sep 4, 2015

Thank you so much)

@meaku
Copy link
Member

meaku commented Sep 4, 2015

No worries! I just had to try the whole thing, because i'm not into webpack as deeply as @jhnns ;)

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

Successfully merging this pull request may close these issues.

3 participants