8000 refactor: defer resolution of markdown-it by kfei · Pull Request #1 · macedigital/angular-markdown-it · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

refactor: defer resolution of markdown-it #1

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

Merged
merged 1 commit into from
May 19, 2016

Conversation

kfei
Copy link
Contributor
@kfei kfei commented May 11, 2016

Thank you for this repo. 😄

I'd like to suggest a small patch to defer the resolution of markdown-it so that:

  1. markdown-it can be lazy-loaded
  2. we can put the <script> tag of markdown-it somewhere after
    angular-markdown-it

Signed-off-by: kfei kfei@kfei.net

1) So markdown-it can be lazy-loaded
2) So we can put the <script> tag of markdown-it somewhere after
   angular-markdown-it

Signed-off-by: kfei <kfei@kfei.net>
@codecov-io
Copy link

Current coverage is 92.86%

Merging #1 into master will increase coverage by -7.14%

@@           master         #1   diff @@
========================================
  Files           1          1          
  Lines          24         28     +4   
  Methods         0          0          
  Branches        0          0          
========================================
+ Hits           24         26     +2   
- Misses          0          2     +2   
  Partials        0          0          

Powered by Codecov. Last updated by a83e3a6...3dc7a08

@macedigital
Copy link
Owner

@kfei Sorry for the late reply, I really like the idea 👍
Let me have a look later in the evening

@macedigital
Copy link
Owner

It took me a while, but yeah, lets do this 😄
The change will go live with the next release.

@macedigital macedigital merged commit 4a99ebe into macedigital:master May 19, 2016
@kfei
Copy link
Contributor Author
kfei commented May 20, 2016

@macedigital Thanks for your time 😄

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
0