From b6a585a6467f833d1fda4584dc956649358b9dd7 Mon Sep 17 00:00:00 2001 From: Scott Nonnenberg Date: Mon, 7 May 2018 19:08:45 -0700 Subject: [PATCH] MessageView: Add id from database to DOM for outgoing messages (#2350) --- js/views/message_view.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/js/views/message_view.js b/js/views/message_view.js index 0cd3e06ae..f3ea8a8e0 100644 --- a/js/views/message_view.js +++ b/js/views/message_view.js @@ -313,6 +313,7 @@ onChange() { this.renderSent(); this.renderQuote(); + this.addId(); }, select(e) { this.$el.trigger('select', { message: this.model }); @@ -470,6 +471,13 @@ return body || isGroupUpdate || isEndSession || errorsCanBeContents; }, + addId() { + // Because we initially render a sent Message before we've roundtripped with the + // database, we don't have its id for that first render. We do get a change event, + // however, and can add the id manually. + const { id } = this.model; + this.$el.attr('id', id); + }, render() { const contact = this.model.isIncoming() ? this.model.getContact() : null; const attachments = this.model.get('attachments');