Skip to content

Commit

Permalink
fix: correct notification deletion logic
Browse files Browse the repository at this point in the history
[skip ci]
  • Loading branch information
Kyle Clemens committed Jul 14, 2018
1 parent 7ec517f commit abc9217
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 10 deletions.
4 changes: 2 additions & 2 deletions webserver/web/src/js/notifications.js
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
(function() {
for (const button of document.querySelectorAll('.message > .message-header > .delete')) {
button.addEventListener('click', () => this.parentElement.parentElement.remove());
button.addEventListener('click', e => e.currentTarget.parentElement.parentElement.remove());
}
for (const button of document.querySelectorAll('.notification > .delete')) {
button.addEventListener('click', () => this.parentElement.remove());
button.addEventListener('click', e => e.currentTarget.parentElement.remove());
}
})();
10 changes: 4 additions & 6 deletions webserver/web/static/js/notifications.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion webserver/web/static/js/notifications.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion webserver/web/templates/base.html.tera
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
<script
defer
src="/static/js/notifications.js?v={{ resources_version }}"
integrity="sha384-SpOHsKR6B3ZYO5uf/rzzNY1FKy8ocZINBHMcGUbYNOOTftqNAg25Me3hYeIlLCr3"></script>
integrity="sha384-at8m89v1aeryT/Wwgi4wPHeFlkzND+DAkWMR+p5tcYE6VJuMtSbKsbjQPlSrAYfr"></script>
<link
rel="stylesheet"
href="/static/css/fontawesome/solid.css?v={{ resources_version }}"
Expand Down

0 comments on commit abc9217

Please sign in to comment.