EVOgeek
/
Vmeda.Online
1
0
Fork 1
Code Issues Pull Requests Packages Projects Releases Wiki Activity

MOBILE-3330 messages: Fix scroll to bottom when receiving messages

main
Dani Palou 2020-03-02 14:40:50 +01:00
parent 7dee04764a
commit 06b8e50f09
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/addon/messages/pages/discussion/discussion.ts
View File

@ -366,9 +366,12 @@ export class AddonMessagesDiscussionPage implements OnDestroy {
return; return;
} }
// Don't use domUtils.getScrollHeight because it gives an outdated value after receiving a new message.
const scrollHeight = this.content && this.content.getScrollElement() ? this.content.getScrollElement().scrollHeight : 0;
// Check if we are at the bottom to scroll it after render. // Check if we are at the bottom to scroll it after render.
// Use a 5px error margin because in iOS there is 1px difference for some reason. // Use a 5px error margin because in iOS there is 1px difference for some reason.
this.scrollBottom = Math.abs(this.domUtils.getScrollHeight(this.content) - this.domUtils.getScrollTop(this.content) - this.scrollBottom = Math.abs(scrollHeight - this.domUtils.getScrollTop(this.content) -
this.domUtils.getContentHeight(this.content)) < 5; this.domUtils.getContentHeight(this.content)) < 5;
if (this.messagesBeingSent > 0) { if (this.messagesBeingSent > 0) {