Skip to content

Conversation

@AlexKaiqi
Copy link
Contributor

I found there is reference error when referring to ../stylesheets/*. Maybe this is because you have changed your directory to ../style. I fixed three pages, and this is all I can find. Hope this fix is correct.

fix ../stylesheets/images/bg-ico.png reference to ../style/images/bg-ico.png
fix ../stylesheets/images/bg-ico.png to ../style/images/bg-ico.png
fix ../stylesheets/images/bg-ico.png to ../style/images/bg-ico.png
@AlexKaiqi
Copy link
Contributor Author

I found this is a great theme for me, it is simple and elegant. Thank you.

It seems that you forgot to add <head> tag.
It seems you missed a </div> tag at the end. But I am not quite sure about this, due to its complexity. 
Please be careful when try to pass this pull request.
I also find {{ page.content | strip_html | strip_newlines | truncate: 120 }} in your head.html does not work,
it translated to something like {% for post in site.posts %}    {% capture this_year %}{{ post.date | date: "%Y" }}{% endcapture %}    {% capture...". I tried to replace page.content variable, and it works. So I guess it is probably because page.content contains other liquid
syntax. But i don't know how to fix it now
@Liberxue Liberxue merged commit 161be0f into Liberxue:master Oct 28, 2017
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.

2 participants