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

Merge pull request #2309 from dpalou/MOBILE-3330 

MOBILE-3330 messages: Fix scroll to bottom when receiving messages
main
Juan Leyva 2020-03-09 14:07:30 +01:00 committed by GitHub
parent 050e7f643a 06b8e50f09
commit eaebca6d2a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
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;
}
// 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.
// 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;
if (this.messagesBeingSent > 0) {