Cell moves up when output comes

[jrieken]

• see recording
• no clear steps but I believe this happens when running a cell that isn't selected

Screen.Recording.2021-04-20.at.12.20.59.mov

[rebornix]

I have a feeling it's related to #121140 (comment)

[rebornix]

Previously we try to make sure the visual view position of the selected cell not change at all, but in this particular scenario, the previous cell gets executed and the cell height grows after outputs are rendered. I pushed a change to improve this and the current behavior is

• if the cell gets executed is visible in the viewport (its top border is in the viewport), then when it grows, the view won't scroll, not matter if the focused element is in the viewport or not
• if the cell gets executed is partially hidden (the top border is hidden), then we will try to ensure the focus element's position won't change

[jrieken]

Better, but not gone...

up.mov

[roblourens]

I think this can be closed. Cell gets focused when executed and its scroll position is maintained

[gilles-zerah]

Note that Ctrl+Enter does not move the cell, from my experience, but naturally keeps the focus on the current cell.
I guess this is understandable since there is no change of focus contrary to Shift+Enter