-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
make spliting of blocks for any markdown content #19
base: main
Are you sure you want to change the base?
Conversation
11bc391
to
2efdff7
Compare
3da1bec
to
0adfc12
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you use else if
if its possible. and also keep the shorter if cases together
plugin/awesoMD/plugin.js
Outdated
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} | |
inCodeBlock = !inCodeBlock | |
return | |
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missed 1 line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, we can do it but lets make the code more readable. It will be easy to understand the code.
0adfc12
to
494bd53
Compare
221004b
to
1777331
Compare
1777331
to
d8aca20
Compare
Description
When code block having empty line within the block, was added in the same slide with alerts, then the blocks were not rendered as expected. Likewise, with other markdown contents. So, proper splitting need to be done. So, with this PR the
splitSlideContentIntoBlocks
function is refined to support different markdown contents.spitSlideContentIntoBlocks
functionality to split all markdown contentsplitSlideContentIntoBlocks