Skip to content
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

Scrolling not working if messages are covered with container #83

Open
avfirsov opened this issue Jun 3, 2020 · 6 comments
Open

Scrolling not working if messages are covered with container #83

avfirsov opened this issue Jun 3, 2020 · 6 comments
Assignees
Labels
Milestone

Comments

@avfirsov
Copy link

avfirsov commented Jun 3, 2020

Describe the bug
My problem and solution are clearly described here: https://stackoverflow.com/questions/50455751/vue-chat-scroll-doesnt-work-with-a-loader

A working demo reproducing the problem: https://jsfiddle.net/StpFlp_DDK/6s0kbtkr/

Why it is matter
In my case, I have a distinct categories for read and unread messages, each having separate container. And when new unread messages arrive, they attached to the unread messages container. Thus, the change in elements count is not detected now and scrolling is not working in this case. I think it is a common practice to split chat messages into categories or groups, not extactly into read and unread, but maybe groupping them by send date, and if new message arrived e.g. not later than 5 minute after last one, it is attached to last group. Or consider groupping by sender. Just look at any modern IM, like Telegram, WhatsApp, etc.. - they all use some or even several types of groupping messages at once.

@theomessin
Copy link
Owner

Version 2 should solve this issue as it'll be using a ResizeObserver (see #72).

@theomessin theomessin added feature and removed bug labels Jun 7, 2020
@theomessin theomessin added this to the v2.0 milestone Jun 7, 2020
@vortechron
Copy link

@theomessin when ver 2 will release?

@theomessin
Copy link
Owner

theomessin commented Jul 9, 2020

@vortechron Soon™.
This is an open-source project, feel free to contribute to the alpha branch 😉.
Too busy flying at the moment 🛩️

@vortechron
Copy link

@theomessin sry, just asking.. tq for the awesome package 👍

@vesper8
Copy link

vesper8 commented Sep 11, 2020

@theomessin would you say that the alpha branch is currently usable? I can imagine the warning of "don't use this in production" but I am in the same situation as the commenters in this issue so.. does the alpha branch currently resolve this issue?

Thanks! Also really looking forward to v2 : )

@theomessin
Copy link
Owner

Hi @vesper8. I'll try to get some v2 work done this week. I'd avoid using the alpha just yet. The next alpha will have massive breaking changes as I'll be switching to ResizeObservers (this will mean you have to change your html layout to support v-chat-scroll. As always, yes you can use it - you just might run into issues. I'll let you know when the new alpha is out which will solve this issue.

Repository owner deleted a comment from saltybuckets Oct 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants