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

Weird spacing in bridged chats #358

Closed
ollietheotter983 opened this issue Oct 1, 2024 · 4 comments · Fixed by #366
Closed

Weird spacing in bridged chats #358

ollietheotter983 opened this issue Oct 1, 2024 · 4 comments · Fixed by #366

Comments

@ollietheotter983
Copy link

Describe the bug
Messages are seperated by icon and nametag from the sender in bridged chats (i cannot word this well so the pics should hopefully help)

To Reproduce
Steps to reproduce the behavior:

  1. Go to a bridged chat (e.g. whatsapp chat) in Commet
  2. Send a few messages
  3. See it split them up as if there was something between

Expected behavior
See screenshots

Screenshots
Current result:
Screenshot_20240930_211104_Commet.jpg

How it should group (in Element bc it works):

Screenshot_20240930_211136_Element.jpg

Additional context
I feel like it could be solved by just running like a one minute timer or so on message send, and grouping the messages together if its within the timer regardless of who sends it as long as there isnt an interruption by someone else

Device Information

Device
Platform: android
Version: v0.3.0-prerelease+2973942
Git Hash: 2973942d27cd36c305687a5cbcdbd8ac35b17df9
Detail: google_services

System Info

Product: a34xdxx

@Airyzz
Copy link
Contributor

Airyzz commented Oct 8, 2024

My suspicion is that there are unknown event types being sent in between those two messages, which dont get rendered but do interrupt the normal spacing. If you enable developer mode, it will render unknown events and you can check

@ollietheotter983
Copy link
Author

ollietheotter983 commented Oct 9, 2024 via email

@ollietheotter983
Copy link
Author

Screenshot_20241009_085223_Commet.jpg

@ollietheotter983
Copy link
Author

i guess the solution would be make the spacing ignore what isnt visible?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants