2

I've been postponing fixing this error that I have been having for a while now. I have the below chatwindow:

enter image description here

The window where I display the messages is a separate component (chat-window.component.ts). I want to scroll to the bottom with ngOnChanges.

When we receive the conversation with the messages from the parent component, where it is received from the server via an asynchronous request, we want to scroll to the bottom of the window element. We do this by calling the this.scrollToBottom() method of the class in the ngOnChanges lifecycle hook.

This.scrollToBottom does get called, but it doesn't scroll to the bottom of the element. Can someone see why?

chat-window.component.ts: in ngOnchanges we do some synchronous stuff before we call this.scrollToBottom()

export class ChatboxWindowComponent implements OnChanges, OnInit, AfterViewChecked {

  @Input('conversation') conversation;
  @ViewChild('window') window;

  constructor() { }

  ngOnChanges() {
    // If the date separators have already been added once, we avoid doing it a second time
    const existingDateObj = this.conversation.messages.findIndex((item, i) => item.dateObj);

    if (existingDateObj === -1) {
      this.conversation.messages.forEach( (item, index, array) => {
        if (index !== 0) {
          const date1 = new Date(array[index - 1].date);
          const date2 = new Date(item.date);

          if (date2.getDate() !== date1.getDate() || date2.getMonth() !== date1.getMonth()) {
            this.conversation.messages.splice(index, 0, {date: date2, dateObj: true});
            console.log(this.conversation.messages.length);
          }
        }
      });
    }

    this.scrollToBottom();
  }

  ngOnInit() {
  }

  ngAfterViewChecked() {
  }

  isItMyMsg(msg) {
    return msg.from._id === this.conversation.otherUser.userId;
  }

  scrollToBottom() {
    try {
      console.log('scrollToBottom called');
      this.window.nativeElement.top = this.window.nativeElement.scrollHeight;
    } catch (err) {}
  }
}

chat-window.component.html

<div #window class="window">
  <ng-container *ngFor="let message of conversation.messages">
    <div class="date-container" *ngIf="!message.msg; else windowMsg">
      <p class="date">{{message.date | amDateFormat:'LL'}}</p>
    </div>
    <ng-template #windowMsg>
      <p
        class="window__message"
        [ngClass]="{
    'window__message--left': isItMyMsg(message),
    'window__message--right': !isItMyMsg(message)
    }"
      >
        {{message.msg}}
      </p>
    </ng-template>
  </ng-container>
</div>
tilly
  • 2,229
  • 9
  • 34
  • 64
  • I forgot to add the template. Up in a second – tilly Nov 02 '18 at 18:23
  • 2
    Try using `scrollTop` instead of `top` like so `this.window.nativeElement.scrollTop = this.window.nativeElement.scrollHeight` – Suryan Nov 02 '18 at 18:28
  • I edited it. It didn't solve the problem. I did 4 console.logs to see what they gave me. I did two (of scrollTop and scrollheight) before I did `this.window.nativeElement.scrollTop = this.window.nativeElement.scrollHeight` and two after. The results before were scrollTop is 0 and scrollHeight is 229. – tilly Nov 02 '18 at 18:40
  • The results after `this.window.nativeElement.scrollTop = this.window.nativeElement.scrollHeight` are scrollTop is still 0 and scrollHeight is still 229 obviously – tilly Nov 02 '18 at 18:41
  • So for some reason it doesn't change the value of this.window.nativeElement.scrollTop to the value of scrollheight. – tilly Nov 02 '18 at 18:42
  • 1
    For a test: does it work if you scroll after a delay: `setTimeout(() => { this.scrollToBottom(); }, 500);`? – ConnorsFan Nov 02 '18 at 18:43
  • 1
    Hope this link would help you out https://stackoverflow.com/questions/35232731/angular2-scroll-to-bottom-chat-style/45367387 – Suryan Nov 02 '18 at 18:45
  • Hey Connor, it worked! How come I had to do this though? – tilly Nov 02 '18 at 18:48
  • I think Suryan's link might explain it. – tilly Nov 02 '18 at 18:49
  • Thanks for all the help guys. – tilly Nov 02 '18 at 18:49

3 Answers3

9

The scroll doesn't work because the list of messages is not rendered yet when you call scrollToBottom. In order to scroll once the messages have been displayed, set a template reference variable (e.g. #messageContainer) on the message containers:

<ng-container #messageContainer *ngFor="let message of conversation.messages">
  ...
</ng-container>

In the code, you can then access these elements with ViewChildren and scroll the window when the QueryList.changes event is triggered:

@ViewChildren("messageContainer") messageContainers: QueryList<ElementRef>;

ngAfterViewInit() {
  this.scrollToBottom(); // For messsages already present
  this.messageContainers.changes.subscribe((list: QueryList<ElementRef>) => {
    this.scrollToBottom(); // For messages added later
  });
}
ConnorsFan
  • 70,558
  • 13
  • 122
  • 146
2

You can add the following code into your HTML element.

#window [scrollTop]="window.scrollHeight" *ngIf="messages.length > 0"

Full code according to your code sample as follows,

<div #window [scrollTop]="window.scrollHeight" *ngIf="messages.length > 0" class="window">
  <ng-container *ngFor="let message of conversation.messages">
    <div class="date-container" *ngIf="!message.msg; else windowMsg">
      <p class="date">{{message.date | amDateFormat:'LL'}}</p>
    </div>
    <ng-template #windowMsg>
      <p
        class="window__message"
        [ngClass]="{
    'window__message--left': isItMyMsg(message),
    'window__message--right': !isItMyMsg(message)
    }"
      >
        {{message.msg}}
      </p>
    </ng-template>
  </ng-container>
</div>

This is work for me. (Currently, I'm using Angular 11)

Tharindu Lakshan
  • 3,995
  • 6
  • 24
  • 44
0

You can use this code

<div id="focusBtn"></div>

const element = document.getElementById("focusBtn");
element.scrollIntoView({ behavior: "smooth", block: "end", inline: "nearest" });
Mukul Raghav
  • 349
  • 2
  • 5