Improve window auto scrolling behavior

[mdiehl216]

When an input cell is just at the bottom of the window, the standard behavior after hitting enter is that the window scrolls up automatically. But it scrolls up so far that the previous input and its output are out of vision.

This often undesirable. After hitting enter, I usually want to see the output.

I use Windows OS.

Desirable is that the window scrolls up so far that either the last input cell or at least its output are visible at the top of the window.

Thank you!

[robert-dodier]

Thank you for the bug report. Looks like this problem is related to #1939.

[mdiehl216]

Yes. It seems to me that in both bug reports the scrolling is indicated correctly, only the final scrolling position is wrong.

[gunterkoenigsmann]

The problem is that I never found out how scrolling actually works...

[robert-dodier]

git grep scroll shows that scrolling is mentioned in various places in the code base; the problem is just to identify the place or places where scrolling is being carried out in order to expose stuff at the bottom of the window, and make the scroll distance smaller (e.g. scroll only enough to make the new cell visible). I will try to look at that in the near future although if someone else can cover it, that would be terrific.

[mdiehl216]

Since the related bug mentions the scrolling problem happening at long output, it may be that there is a test that measures if the current output fits into the current window. If not, the window is scrolled. So the problem is not necessarily that the cell is at the bottom of the window. In both cases (long output, cell at bottom of the window) scrolling is necessary. It's just that the final position is wrong.

…

On Sat, 7 Sep 2024, 05:20 Robert Dodier, ***@***.***> wrote: git grep scroll shows that scrolling is mentioned in various places in the code base; the problem is just to identify the place or places where scrolling is being carried out in order to expose stuff at the bottom of the window, and make the scroll distance smaller (e.g. scroll only enough to make the new cell visible). I will try to look at that in the near future although if someone else can cover it, that would be terrific. — Reply to this email directly, view it on GitHub <#1947 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BHQAVV6PP4HWMUFSFWI2MXTZVIMABAVCNFSM6AAAAABNNWFSNGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZUHAZDKNRUG4> . You are receiving this because you authored the thread.Message ID: ***@***.***>