-
-
Notifications
You must be signed in to change notification settings - Fork 314
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
Add CoilImagesPlugin Module #174
Conversation
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.
Very good job! 🙌
@@ -0,0 +1,35 @@ | |||
# Image Coil |
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.
😍 ! Very nice touch to include documentation!
docs/docs/v4/image-coil/README.md
Outdated
|
||
Image loading based on `Coil` library | ||
|
||
```kt |
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.
Unfortunately syntax highlighting won't work for kt
, please change to kotlin
import io.noties.markwon.image.ImageSpanFactory; | ||
|
||
/** | ||
* @since 4.0.0 |
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.
You can put here some attribution if you wish, like your name and email, but it's optional
Closes #166
This adds support for loading images using Coil through a new
markwon-image-coil
module based off the implementations of the existing image plugins. I'm not too familiar with any gotchas there would be with interactions between the Markwon and Coil APIs so any guidance would be much appreciated!