Skip to content

fix: scroll to bottom works also for long messages - #19018

Open
max65482 wants to merge 1 commit into
nextcloud:mainfrom
max65482:fix/scroll-to-bottom-race-condition
Open

fix: scroll to bottom works also for long messages#19018
max65482 wants to merge 1 commit into
nextcloud:mainfrom
max65482:fix/scroll-to-bottom-race-condition

Conversation

@max65482

Copy link
Copy Markdown
Contributor

☑️ Resolves

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@max65482
max65482 force-pushed the fix/scroll-to-bottom-race-condition branch 2 times, most recently from 61ac5eb to 0ed4fd2 Compare August 16, 2026 18:28

@Antreesy Antreesy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contribution!

Analysis with race condition is straight to the point, but solution is overcomplicated. Please adress review comments, and it will be ready to merge/backport

Comment thread src/components/MessagesList/MessagesList.vue Outdated
Comment thread src/composables/useGetMessages.ts Outdated
Comment thread src/composables/useGetMessages.ts Outdated
@Antreesy Antreesy added this to the ⛅ Next Beta/RC (35) milestone Aug 17, 2026
@Antreesy Antreesy added bug feature: chat 💬 Chat and system messages feature: frontend 🖌️ "Web UI" client labels Aug 17, 2026
@max65482
max65482 force-pushed the fix/scroll-to-bottom-race-condition branch from 0ed4fd2 to 9b90abc Compare August 17, 2026 15:48
Signed-off-by: Maximilian Martin <maximilian_martin@gmx.de>
@max65482
max65482 force-pushed the fix/scroll-to-bottom-race-condition branch from 9b90abc to 446f598 Compare August 17, 2026 15:53
@max65482

Copy link
Copy Markdown
Contributor Author

Implemented your suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug feature: chat 💬 Chat and system messages feature: frontend 🖌️ "Web UI" client

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can not scroll to end with "Scroll to bottom" on long messages

2 participants