Merge pull request #14266 from nextcloud/fix/noid/message-list-sticky

Author: Antreesy

src/components/CallView/shared/VideoVue.spec.js

@@ -14,12 +14,6 @@ import storeConfig from '../../../store/storeConfig.js'
 import EmitterMixin from '../../../utils/EmitterMixin.js'
 import CallParticipantModel from '../../../utils/webrtc/models/CallParticipantModel.js'
 
-global.ResizeObserver = jest.fn().mockImplementation(() => ({
-	observe: jest.fn(),
-	unobserve: jest.fn(),
-	disconnect: jest.fn(),
-}))
-
 describe('VideoVue.vue', () => {
 	let localVue
 	let store

src/components/MessagesList/MessagesList.vue

@@ -341,6 +341,10 @@ export default {
 		subscribe('networkOnline', this.handleNetworkOnline)
 		window.addEventListener('focus', this.onWindowFocus)
 
+		window.addEventListener('resize', this.updateSize)
+		this.resizeObserver = new ResizeObserver(this.updateSize)
+		this.resizeObserver.observe(this.$refs.scroller)
+
 		/**
 		 * Every 30 seconds we remove expired messages from the store
 		 */
@@ -364,6 +368,11 @@ export default {
 		unsubscribe('networkOffline', this.handleNetworkOffline)
 		unsubscribe('networkOnline', this.handleNetworkOnline)
 
+		window.removeEventListener('resize', this.updateSize)
+		if (this.resizeObserver) {
+			this.resizeObserver.disconnect()
+		}
+
 		if (this.expirationInterval) {
 			clearInterval(this.expirationInterval)
 			this.expirationInterval = null
@@ -373,6 +382,14 @@ export default {
 	methods: {
 		t,
 		n,
+		updateSize() {
+			if (this.isChatScrolledToBottom) {
+				this.$refs.scroller.scrollTo({
+					top: this.$refs.scroller.scrollHeight,
+				})
+			}
+		},
+
 		prepareMessagesGroups(messages) {
 			let prevGroupMap = null
 			const groupsByDate = {}

src/test-setup.js

@@ -109,6 +109,12 @@ global.BroadcastChannel = jest.fn(() => ({
 	addEventListener: jest.fn(),
 }))
 
+global.ResizeObserver = jest.fn(() => ({
+	observe: jest.fn(),
+	unobserve: jest.fn(),
+	disconnect: jest.fn(),
+}))
+
 // Work around missing "URL.createObjectURL" (which is used in the code but not
 // relevant for the tests) in jsdom: https://github.com/jsdom/jsdom/issues/1721
 window.URL.createObjectURL = jest.fn()