Merge pull request #1572 from vhf/feed-tag-fix

Fix HTML issue with RSS/Atom feeds: rel="alternate"
hexojs · Oct 30, 2015 · f7b204d · f7b204d
2 parents cd0d987 + 3e4a7da
commit f7b204d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/plugins/helper/feed_tag.js b/lib/plugins/helper/feed_tag.js
@@ -6,7 +6,7 @@ function feedTagHelper(path, options) {
  var title = options.title || this.config.title;
  var type = options.type || 'atom';
 
- return '<link rel="alternative" href="' + this.url_for(path) + '" title="' + title + '" type="application/' + type + '+xml">';
+ return '<link rel="alternate" href="' + this.url_for(path) + '" title="' + title + '" type="application/' + type + '+xml">';
 }
 
 module.exports = feedTagHelper;
diff --git a/test/scripts/helpers/feed_tag.js b/test/scripts/helpers/feed_tag.js
@@ -15,14 +15,14 @@ describe('feed_tag', function() {
  var feed = require('../../../lib/plugins/helper/feed_tag').bind(ctx);
 
  it('path', function() {
- feed('atom.xml').should.eql('<link rel="alternative" href="/atom.xml" title="Hexo" type="application/atom+xml">');
+ feed('atom.xml').should.eql('<link rel="alternate" href="/atom.xml" title="Hexo" type="application/atom+xml">');
  });
 
  it('title', function() {
- feed('atom.xml', {title: 'RSS Feed'}).should.eql('<link rel="alternative" href="/atom.xml" title="RSS Feed" type="application/atom+xml">');
+ feed('atom.xml', {title: 'RSS Feed'}).should.eql('<link rel="alternate" href="/atom.xml" title="RSS Feed" type="application/atom+xml">');
  });
 
  it('type', function() {
- feed('rss.xml', {type: 'rss'}).should.eql('<link rel="alternative" href="/rss.xml" title="Hexo" type="application/rss+xml">');
+ feed('rss.xml', {type: 'rss'}).should.eql('<link rel="alternate" href="/rss.xml" title="Hexo" type="application/rss+xml">');
  });
 });