arno/isso
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

embed: fix feed link creation on older browsers

Browse Source 
When a browser doesn't support DOM manipulation convenience methods,
the addition of the feed link was triggering an error because elements
created by `$.new()` are regular elements, not elements from our own
mini-DOM implementation. Therefore, the `append()` method may be
absent. Use `appendChild()` instead.
This commit is contained in:
Vincent Bernat 2018-04-29 10:37:00 +02:00
parent c26d59de08
commit 311f65d52b
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
isso/js/embed.js
View File

@ -31,7 +31,7 @@ require(["app/lib/ready", "app/config", "app/i18n", "app/api", "app/isso", "app/
var feedLink = $.new('a', i18n.translate('atom-feed'));
var feedLinkWrapper = $.new('span.isso-feedlink');
feedLink.href = api.feed($("#isso-thread").getAttribute("data-isso-id"));
feedLinkWrapper.append(feedLink);
feedLinkWrapper.appendChild(feedLink);
$("#isso-thread").append(feedLinkWrapper);
}
$("#isso-thread").append($.new('h4'));