-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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 root path configure for img tag #1444
Conversation
This pull request only fix the image tag. |
5753e49
to
570b46b
Compare
I test run the test with iojs using the master branch, it gets the same error, I think this may be related to the recent upgrade of iojs. |
570b46b
to
4ad77fe
Compare
Add one more commit to fix the problem, when the root path is set to a subdirectory, there is no this root path in the permalink of post. |
69a0e2e
to
30d1953
Compare
30d1953
to
a11d7db
Compare
Current img tag doesn't prepend root path if it's not a slash. With this fix, the root path will be prepend to the img src if it's not a slash. This fix only works for the hexo tags, does not work for the markdown syntax of links and images. The issues of markdown links and images should be fixed in the hexo-renderer-marked module. Resolves: hexojs#1440
d6dd70c
to
b801915
Compare
This commit fixs this problem, when the root path is set to a subdirectory, there is no this root path in the permalink of post.
b801915
to
657f9e7
Compare
I rebased the code on the latest master, And it has the same logic as this pull request: hexojs/hexo-renderer-marked#7 @Xuanwo please check it. |
👍 |
Support root path configure for img tag
Current img tag doesn't prepend root path if it's not a slash. With
this fix, the root path will be prepend to the img src if it's not a
slash.
This fix only works for the hexo tags, does not work for the markdown
syntax of links and images. The issues of markdown links and images should
be fixed in the hexo-renderer-marked module.
Resolves: #1440