signalapp / Signal-Desktop

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

The scrolling steps on the input field are too large by about one line height #6805

Open Meteor0id opened 7 months ago

Meteor0id commented 7 months ago

Using a supported version?

Overall summary

When expanding the input field and scrolling the text jumps up or down so far that not a single line from the previous position stays in view. This makes it uncertain if you accidentally scrolled to far and creates the need to scroll manually with the cursor to double check your position. It's common practice to jump less than 100% so at least one line of a previous position stays in view.

Steps to reproduce

expand input field, fill with large amount of text and scroll

Expected result

keep one line of previous positon in view of new position

Actual result

scrolling is about 100% not leaving any previously seen text in view

Screenshots

No response

Signal version

7.1.0-beta1`

Operating system

Windows 10

Version of Signal on your phone

No response

Link to debug log

No response

trevor-signal commented 7 months ago

@Meteor0id thanks for the report. Can you provide a bit more information about how you are scrolling so I can reproduce the issue? E.g. via keyboard, trackpad, mouse?

Meteor0id commented 7 months ago

Mousewheel. I can imagine the driver or OS determine the step size by default but maybe it's Signal defining this. If possible Signal should attempt to force step size to either 1 line or the number of lines that fit on the current input -1

trevor-signal commented 7 months ago

@Meteor0id thanks, we'll take a look!