-
Notifications
You must be signed in to change notification settings - Fork 28
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
Support file:
URLs (and absolute paths) for images
#92
Comments
Images are already working, including hover text. I’ve made sure that a demo of that is in the test files as well. If you have an example where this syntax doesn’t work, please feel free to reopen this issue with details to reproduce. |
Just tested, you're right: images work for web images. Not for local images though, and I'm using this for pretty markdown rendering of personal notes on my local filesystem. e.g.
|
@Cimbali looks like I can't reopen the issue if it was closed by you, but I've commented the reproduction above and changed the issue title |
Interesting. This seems to work for relative paths, but not with full |
Relative paths works for me as a workaround! Thanks |
file:
URLs (and absolute paths) for images
I think we could rewrite all local paths as relative, specifically only for injecting in local pages. V2 (#99) already has some src/href rewriting so we can just extend that. Not sure that it would work across drive letters on windows. Also this feels like something that could be exploited by malign extensions to probe the filesystem, so I wouldn't rely on it. Firefox may disable this workaround at any point. |
The markdown reference site lists the common markdown syntax for images as the same as a link, i.e.
but with a preceding !, e.g.
This should be relatively easy to implement: when matching that pattern in the source markdown, inject an HTML
<img>
tag to the rendered HTML.When I get time I plan on working on this.
The text was updated successfully, but these errors were encountered: