diff options
| -rw-r--r-- | js/Article.js | 10 | ||||
| -rwxr-xr-x | js/Headlines.js | 11 |
2 files changed, 12 insertions, 9 deletions
diff --git a/js/Article.js b/js/Article.js index a11004a0a..21973518c 100644 --- a/js/Article.js +++ b/js/Article.js @@ -350,15 +350,7 @@ const Article = { Notify.close(); dialog.hide(); - if (data) { - if (Headlines.headlines[data.id]) { - Headlines.headlines[data.id].tags = data.tags; - } - - App.findAll(`span[data-tags-for="${data.id}"`).forEach((ctr) => { - ctr.innerHTML = Article.renderTags(data.id, data.tags); - }); - } + Headlines.onTagsUpdated(data); } catch (e) { App.Error.report(e); } diff --git a/js/Headlines.js b/js/Headlines.js index f98b3dcce..9bc5747c2 100755 --- a/js/Headlines.js +++ b/js/Headlines.js @@ -1240,6 +1240,17 @@ const Headlines = { } } }, + onTagsUpdated: function (data) { + if (data) { + if (this.headlines[data.id]) { + this.headlines[data.id].tags = data.tags; + } + + App.findAll(`span[data-tags-for="${data.id}"`).forEach((ctr) => { + ctr.innerHTML = Article.renderTags(data.id, data.tags); + }); + } + }, // TODO: maybe this should cause article to be rendered again, although it might cause flicker etc onLabelsUpdated: function (data) { if (data) { |