Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix rss link in <head> #10

Closed
wants to merge 3 commits into from
Closed

Fix rss link in <head> #10

wants to merge 3 commits into from

Conversation

yqw1218
Copy link
Contributor

@yqw1218 yqw1218 commented Feb 8, 2015

Rss link in doesn't work while theme.rss is empty.

Move code

{% if theme.rss !== false %}        
  {% if theme.rss %}        
    {% set feed_uri = theme.rss %}      
  {% elif config.feed and config.feed.path %}       
    {% set feed_uri = config.root + config.feed.path %}     
  {% endif %}       
{% endif %}

from header.swig to head.swig, at the same time, simplify the code:

{% if theme.rss !== false and config.feed and config.feed.path %}
    {% set theme.rss = config.root + config.feed.path %}
{% endif %}

@yqw1218 yqw1218 closed this Feb 8, 2015
@yqw1218
Copy link
Contributor Author

yqw1218 commented Feb 8, 2015

手滑……将一个有问题的pull了……请忽略（貌似不能删除？

@yqw1218 yqw1218 changed the title ei Fix rss link in <head> Feb 8, 2015
ivan-nginx added a commit that referenced this pull request May 3, 2017
@ivan-nginx
Merge pull request #10 from iissnan/master
7540815 
Update from original.
@ivan-nginx ivan-nginx mentioned this pull request Aug 9, 2017
Closed
wangpei72 pushed a commit to wangpei72/hexo-theme-next that referenced this pull request Sep 17, 2021
@renovate @renovate-bot
Update dependency js-yaml to v3.14.0 (iissnan#10)
57cf2c9 
Co-authored-by: Renovate Bot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@yqw1218