Skip to content

Fetch more messages does not work with multiple chat windows #368

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

Closed
NINI1988 opened this issue Oct 7, 2022 · 3 comments · Fixed by #392
Closed

Fetch more messages does not work with multiple chat windows #368

NINI1988 opened this issue Oct 7, 2022 · 3 comments · Fixed by #392
Labels
bug Something isn't working

Comments

@NINI1988
Copy link
Contributor

NINI1988 commented Oct 7, 2022

Describe the bug

It is not possible to have multiple chat windows on the same page.
When there are multiple chat windows and you scroll up to the fetch more messages, only the first chat window will fetch new messages and will also trigger this event on the second chat window.
In the second chat window there will be no fetch more messages event when you scroll up.

We are using the chat component in multiple popup windows to communicate with multiple users at the same time.

Steps to reproduce

  1. Go to https://playcode.io/vac_multi_chats
  2. Scroll up the second chat window with the name Room 11 and no messages will be fetched
  3. Scroll up the first chat windows Room 10
  4. See that Room 10 and Room 11 will fetch new messages

Expected behavior

Each chat component should behave independent

Screenshots

Here is an example of our website:
grafik

Device (please complete the following information)

  • OS: Windows 11
  • Browser: Firefox 105, Chrome 106
  • Package version: 2.03
@NINI1988 NINI1988 added the bug Something isn't working label Oct 7, 2022
@antoine92190
Copy link
Collaborator

I didn’t think of that. I will try to support multiple chat windows, but if you have any ideas to fix this issue I’m all hears

antoine92190 added a commit that referenced this issue Nov 26, 2022
@daniser
Copy link

daniser commented Oct 4, 2023

@NINI1988 I am sorry, which window manager library did you use (in the screenshot)? I want to make similar setup.

@NINI1988
Copy link
Contributor Author

NINI1988 commented Oct 9, 2023

It's self implemented by adding divs to the body.
Source: https://gitlab.com/foodsharing-dev/foodsharing/-/blob/master/client/src/conv.js

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants