A couple small fixes: quotes, ConversationListItem, MessageDetail
- Load quoted message even with local thumbnail - A bit more space for message in ConversationListItem - Make the message detail screen scrollable
This commit is contained in:
parent
6e4b73ec0c
commit
fdc7c85876
3 changed files with 10 additions and 5 deletions
|
@ -1471,10 +1471,7 @@
|
||||||
}
|
}
|
||||||
|
|
||||||
// 1. Load provided thumbnail
|
// 1. Load provided thumbnail
|
||||||
if (this.loadQuoteThumbnail(message, quote)) {
|
const gotThumbnail = this.loadQuoteThumbnail(message, quote);
|
||||||
this.forceRender(message);
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
// 2. Check to see if we've already loaded the target message into memory
|
// 2. Check to see if we've already loaded the target message into memory
|
||||||
const { author, id } = quote;
|
const { author, id } = quote;
|
||||||
|
@ -1487,6 +1484,13 @@
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Even if we got the thumbnail locall, we wanted to populate the referenced
|
||||||
|
// message so a click can navigate to it.
|
||||||
|
if (gotThumbnail) {
|
||||||
|
this.forceRender(message);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
// We only go further if we need more data for this message. It's always important
|
// We only go further if we need more data for this message. It's always important
|
||||||
// to grab the quoted message to allow for navigating to it by clicking.
|
// to grab the quoted message to allow for navigating to it by clicking.
|
||||||
const { attachments } = quote;
|
const { attachments } = quote;
|
||||||
|
|
|
@ -52,6 +52,7 @@
|
||||||
.message-detail-wrapper {
|
.message-detail-wrapper {
|
||||||
height: calc(100% - 48px);
|
height: calc(100% - 48px);
|
||||||
width: 100%;
|
width: 100%;
|
||||||
|
overflow-y: auto;
|
||||||
}
|
}
|
||||||
|
|
||||||
.discussion-container {
|
.discussion-container {
|
||||||
|
|
|
@ -1955,7 +1955,7 @@
|
||||||
font-size: 13px;
|
font-size: 13px;
|
||||||
line-height: 18px;
|
line-height: 18px;
|
||||||
|
|
||||||
height: 1.2em;
|
height: 1.3em;
|
||||||
overflow: hidden;
|
overflow: hidden;
|
||||||
white-space: nowrap;
|
white-space: nowrap;
|
||||||
text-overflow: ellipsis;
|
text-overflow: ellipsis;
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue