Skip to content

Conversation

NejcZdovc
Copy link
Collaborator

@NejcZdovc NejcZdovc commented Dec 31, 2016

Resolves #350

Now you can write HTML directly into the markdown files.

@NejcZdovc NejcZdovc changed the title Added split code blocks to home page [WIP] Added split code blocks to home page Dec 31, 2016
@NejcZdovc NejcZdovc force-pushed the feature/#350 branch 2 times, most recently from 45b72c7 to 881b47f Compare December 31, 2016 17:08
@bebraw
Copy link
Contributor

bebraw commented Jan 1, 2017

Looks good to me. Do you want to add anything else to the PR?

@bebraw bebraw changed the base branch from develop to master January 1, 2017 13:54
@bebraw
Copy link
Contributor

bebraw commented Jan 1, 2017

Can you rebase against master? The development happens against it now (faster deployments).

@NejcZdovc
Copy link
Collaborator Author

@bebraw rebased

}

codeArray.forEach(item => {
// if item is not code, then check for html tags and parse accordingly
Copy link
Member

@SpaceK33z SpaceK33z Jan 1, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What happens with code in backticks (e.g. `<script>`)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still need to cover this scenario, when we use only single backtick

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Single backtick is now covered as well.

@NejcZdovc
Copy link
Collaborator Author

@SpaceK33z can you look at it again please

@NejcZdovc NejcZdovc changed the title [WIP] Added split code blocks to home page Added split code blocks to home page Jan 9, 2017
Copy link
Member

@SpaceK33z SpaceK33z left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested this out and it works!

@bebraw, could you verify the code is good? I can't really comment on that part. You can merge after that.

@bebraw bebraw merged commit d568ad5 into webpack:master Jan 15, 2017
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