Skip to content

Conversation

@GretaD
Copy link
Contributor

@GretaD GretaD commented Sep 10, 2025

fixes #11646

@GretaD GretaD self-assigned this Sep 10, 2025
@GretaD GretaD marked this pull request as ready for review September 10, 2025 14:26
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

🙈

ok, makes sense. guess we tested with short messages in the past where top/middle makes not a lot of difference?

@ChristophWurst
Copy link
Member

/backport to stable5.4

@GretaD GretaD merged commit 379ea52 into main Sep 11, 2025
45 checks passed
@GretaD GretaD deleted the fix/thread-scroll branch September 11, 2025 08:11
@kesselb
Copy link
Contributor

kesselb commented Oct 9, 2025

This is causing Element.scrollIntoView: 'top' (value of 'block' member of ScrollIntoViewOptions) is not a valid value for enumeration ScrollLogicalPosition when opening a message/thread.

Would you mind opening a follow-up and use a valid value for block https://developer.mozilla.org/en-US/docs/Web/API/Element/scrollIntoView#block?

@ChristophWurst
Copy link
Member

#11674

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stop the scrolling to a middle of a (long) email or thread

4 participants