Skip to content

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