Chatroom scrolling bug

[taoeffect]

Problem

Switching between chatrooms when you've left scrolled to the bottom doesn't show the bottom when you switch back:

scrolling-bug-ff.compressed.mov

Solution

Make sure that you're scrolled all the way down, while still preserving the ability to remember where you were.

If you're far enough down it should scroll you all the way to the bottom.

*Note: implement #2556 before implementing this issue.

[SebinSong]

@taoeffect
Need your help in demystifying one thing re chatroom.js contract. It's height prop of GIMessage (eg. below)

group-income/frontend/model/contracts/chatroom.js

Line 311 in 5b89d70

process ({ direction, data, meta, hash, height, innerSigningContractID }, { state }) {

Q.
From my investigation, it seems like a value we can use to determine if a specific GIMessage was created earlier/later than others. For example in a chatroom context, (It appears) the app uses this value to determine a particular chat-message piece was sent earlier than another (if height of messageA is greater than height of messageB, messageA is more recent message).

reference:

group-income/frontend/views/containers/chatroom/ChatMain.vue

Lines 174 to 182 in 5b89d70

	const bottomMessageCreatedHeight = msg.height
	const latestMessageCreatedHeight = this.currentChatRoomReadUntil?.createdHeight
	// No need to check for pending here as it's checked above
	if (!latestMessageCreatedHeight || latestMessageCreatedHeight <= bottomMessageCreatedHeight) {
	this.updateReadUntilMessageHash({
	messageHash: msg.hash,
	createdHeight: msg.height
	})
	}

Is my understanding correct?

[taoeffect]

if height of messageA is greater than height of messageB, messageA is more recent message

That's correct. 👍

[taoeffect]

Closed in #2598!