From 985c6f34f139726e4c303a4bc91a048b4340555c Mon Sep 17 00:00:00 2001 From: Jules Laplace Date: Wed, 13 Dec 2017 05:20:22 +0100 Subject: yes --- public/assets/js/lib/views/details/settings.js | 2 +- public/assets/js/lib/views/mail/mailbox.js | 24 +++++++++++++----------- 2 files changed, 14 insertions(+), 12 deletions(-) (limited to 'public/assets/js/lib/views') diff --git a/public/assets/js/lib/views/details/settings.js b/public/assets/js/lib/views/details/settings.js index 777a9ba..b702908 100644 --- a/public/assets/js/lib/views/details/settings.js +++ b/public/assets/js/lib/views/details/settings.js @@ -45,7 +45,7 @@ var ThreadSettingsForm = FormView.extend({ option.innerHTML = color $color.append(option) }) - $color.val(thread.color || keyword.color) + $color.val(thread.color || keyword ? keyword.color : "") $.get('/api/keywords', function(data){ var $keyword = this.$('[name=keyword]') diff --git a/public/assets/js/lib/views/mail/mailbox.js b/public/assets/js/lib/views/mail/mailbox.js index b7877f2..22a49c5 100644 --- a/public/assets/js/lib/views/mail/mailbox.js +++ b/public/assets/js/lib/views/mail/mailbox.js @@ -20,19 +20,21 @@ var MailboxView = View.extend({ }, populate: function(data){ - this.boxlist.load(data.boxes) + if (data.boxes) { + this.boxlist.load(data.boxes) - var user = data.user - var max = data.messages.length-1 - if (data.messages.length) { - $("#no_messages").hide() + var user = data.user + var max = data.messages.length-1 + if (data.messages.length) { + $("#no_messages").hide() + } + data.messages.forEach(function(message, i){ + var $row = this.parse(message, user) + if (i === 0) $row.addClass("first") + if (i === max) $row.addClass("last") + this.$el.append($row) + }.bind(this)) } - data.messages.forEach(function(message, i){ - var $row = this.parse(message, user) - if (i === 0) $row.addClass("first") - if (i === max) $row.addClass("last") - this.$el.append($row) - }.bind(this)) $("body").removeClass('loading') }, -- cgit v1.2.3-70-g09d2