Messages bubbles with errors are clickable
For messages with errors, clicking anywhere inside the bubble takes you to the message detail view. // FREEBIE
This commit is contained in:
parent
c79a917bbe
commit
47befdbf61
3 changed files with 14 additions and 11 deletions
|
@ -16,7 +16,8 @@
|
|||
this.listenTo(this.model, 'destroy', this.remove);
|
||||
},
|
||||
events: {
|
||||
'click .timestamp': 'select'
|
||||
'click .timestamp': 'select',
|
||||
'click .error': 'select'
|
||||
},
|
||||
select: function() {
|
||||
this.$el.trigger('select', {message: this.model});
|
||||
|
@ -35,7 +36,7 @@
|
|||
renderErrors: function() {
|
||||
var errors = this.model.get('errors');
|
||||
if (_.size(errors) > 0) {
|
||||
this.$el.addClass('error');
|
||||
this.$('.bubble').addClass('error');
|
||||
if (this.model.isIncoming()) {
|
||||
this.$('.content').text(this.model.getDescription()).addClass('error-message');
|
||||
}
|
||||
|
|
|
@ -262,14 +262,15 @@
|
|||
opacity: 1.0;
|
||||
}
|
||||
|
||||
.error .bubble {
|
||||
.error.bubble {
|
||||
cursor: pointer;
|
||||
background: url('/images/error_red.png') no-repeat;
|
||||
}
|
||||
.incoming.error .bubble {
|
||||
.incoming .error.bubble {
|
||||
padding-right: 40px;
|
||||
background-position: calc(100% - 12px) 9px;
|
||||
}
|
||||
.outgoing.error .bubble {
|
||||
.outgoing .error.bubble {
|
||||
padding-left: 40px;
|
||||
background-position: 12px 9px;
|
||||
}
|
||||
|
|
|
@ -729,15 +729,16 @@ input.search {
|
|||
.message-detail .outgoing.sent .bubble,
|
||||
.message-list .outgoing.sent .bubble {
|
||||
opacity: 1.0; }
|
||||
.message-detail .error .bubble,
|
||||
.message-list .error .bubble {
|
||||
.message-detail .error.bubble,
|
||||
.message-list .error.bubble {
|
||||
cursor: pointer;
|
||||
background: url("/images/error_red.png") no-repeat; }
|
||||
.message-detail .incoming.error .bubble,
|
||||
.message-list .incoming.error .bubble {
|
||||
.message-detail .incoming .error.bubble,
|
||||
.message-list .incoming .error.bubble {
|
||||
padding-right: 40px;
|
||||
background-position: calc(100% - 12px) 9px; }
|
||||
.message-detail .outgoing.error .bubble,
|
||||
.message-list .outgoing.error .bubble {
|
||||
.message-detail .outgoing .error.bubble,
|
||||
.message-list .outgoing .error.bubble {
|
||||
padding-left: 40px;
|
||||
background-position: 12px 9px; }
|
||||
.message-detail .incoming .bubble,
|
||||
|
|
Loading…
Reference in a new issue