New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(client): Broken chats #8951
Conversation
New messages come in live BUT it erases the prev message. Testing on https://stop.voring.me |
@@ -292,6 +292,7 @@ definePageMetadata(computed(() => !fetching ? user ? { | |||
<style lang="scss" scoped> | |||
.mk-messaging-room { | |||
position: relative; | |||
overflow: scroll; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think using overflow-y
here is a better idea. Otherwise browsers will always show an unnecessary scroll bar for horizontal scrolling.
Or maybe use overflow: auto;
instead to let the browser decide when scroll bars are necessary.
The problem of messages being cut off only seems to happen on mobile. Works fine on desktop. |
I think i'm cannot handle with these problems, those are more complicated than i thought. |
I don't think it should be too hard to fix, just a tiny bit of positioning. If you re-open this PR I'd be happy to take a look and see if I can make a solution :) |
What & Why
overflow: scroll
in stylegetScrollContainer(rootEl.value)
is nullAdditional info (optional)