Enable auto scrolling for outputs by default

[jtpio]

Fixes #287

autoscroll-outputs.mp4

TODO

• Replicate the classic notebook logic: https://github.com/jupyter/notebook/blob/a9a31c096eeffe1bff4e9164c6a0442e0e13cdb3/notebook/static/notebook/js/outputarea.js#L96-L120
• Add UI Test
• Open issue / PR to upstream in JupyterLab
  • Collapsing Output cells - Downgrade from Notebook jupyterlab#5897
  • Auto scroll outputs jupyterlab#11760

cc @yuvipanda

[github-actions]

👈 Launch RetroLab on Binder

[jtpio]

Converting to draft for now.

We should check the logic in the classic notebook to handle the auto scrolling.

With the naive logic as it is now in this PR this would enable it even for small outputs:

[yuvipanda]

Trying to match classic makes sense here. I think we should eventually upstream this into lab, but this is a great place to try this out. Thanks a lot for looking into this, @jtpio

[jtpio]

There is some logic in the _should_scroll function here:

https://github.com/jupyter/notebook/blob/a9a31c096eeffe1bff4e9164c6a0442e0e13cdb3/notebook/static/notebook/js/outputarea.js#L96-L120

[jtpio]

A follow-up to this would be to update to JupyterLab 3.2.5 packages to grab jupyterlab/jupyterlab#11478

[jtpio]

Opened jupyterlab/jupyterlab#11760 to track that upstream.

[jtpio]

cc @yuvipanda if you want to give it a try on Binder: https://mybinder.org/v2/gh/jtpio/retrolab/scroll-outputs?urlpath=retro/tree

I'll do a bit more testing but if it works fine we can merge and release.

Thinking we might also want to make it possible to disable that behavior via the settings system (opt-out).

[jtpio]

Thinking we might also want to make it possible to disable that behavior via the settings system (opt-out).

Implemented in 84df24d

[jtpio]

Did some more testing on Binder and it seems to be working fine.

Will make a new release with this change after merging.