signalapp / Signal-Desktop

A private messenger for Windows, macOS, and Linux.
https://signal.org/download
GNU Affero General Public License v3.0
14.69k stars 2.68k forks source link

The entire viewport is jumping because inter-message space re-adjusts after sending #5828

Closed spacesynth closed 2 years ago

spacesynth commented 2 years ago

Bug Description

Because of Signal freeing up extra space between messages the entire chat log bounces first up after posting (intended), then down again profusely as it frees the extra space with a time delay.

Steps to Reproduce

  1. Send message and watch gap between messages
  2. The extra space is freed up with a delay
  3. Watch the entire chatlog bounce TWICE first up and then back down

Actual Result:

Uneeded execution of the script that clears the space after sending the message.

Expected Result:

The extra space should be cleared before the new message even appears, so the chatlog doesn't hop and create motion sickness.

Screenshots

right during sending grafik

seconds after sending grafik

Platform Info

Signal Version:

grafik

Operating System:

grafik

Linked Device Version:

5.33.7

Link to Debug Log

not applicable

I wish I was joking, but it does indeed create sort of a motion sickenss for me. The simple fact that this jumping doesn't occur on phone and only showed after the update indicates to me that it is a bug. It was honestly fine before the update today!

Since the gaps are only meant to appear if there was a significant break in chat activity for several minutes, they are created wrongly and deleted right after sending, leading to this irritating "hopping".

I hope you can bear with me reporting this, it just looks strange and annoying to my brain.

scottnonnenberg-signal commented 2 years ago

Thanks for reporting this. The current purpose for splitting the message out is to make sure that the status of the message can be shown. But we can change it so the new message is grouped but also shows its differing status - that's what the phones do.

spacesynth commented 2 years ago

Thank you for elaborating on this and adding the bug label. Whilst it is only a visual nuisance I am happy you consider addressing this issue.

spacesynth commented 2 years ago

grafik

grafik

It would appear that the jumping gaps are not appearing anymore when sending. Thanks everyone for the very fast fix!