put limits on content in the session pane

[s-u]

Once the session pane accumulates too much output the browser becomes very laggy (it won't respond for many seconds) and will crash eventually. It may be desirable to limit the amount of content in the session pane by flushing old content or some such strategy. It is in particular problematic due to (lack of) #1734 which would at least allow manual intervention.

[gordonwoodhull]

Infinite scrolling.

It would be difficult in the notebook pane but with the completely static content of the session pane it should be easy to test a few libraries and use one.

[shaneporter]

How would infinite scrolling benefit here? New content is automatically appended to the bottom of the session pane, right?

Are you suggesting that as new content is appended, some old content is removed from the pane? And that this content is available to show again, should the user scroll back up?

Infinite scrolling works because it appends content, and the user has to continue scrolling down to see the new content; their current position isn't interrupted by the new content.

Surely scrolling up to see previous content would disorientate the user?

[gordonwoodhull]

There are different implementations of infinite scrolling.

Sure, the cheap way is just to keep appending stuff. But I'm thinking of what data tables and editors do where they only instantiate the DOM elements for the stuff that's on the screen. As you scroll they are adding and removing DOM elements on either side.

In fact, one implementation could be just to use ACE since I know they handle this correctly. There might still need to be some limit but I don't think so. I've pasted multiple copies of moby dick and war and peace into assets before.

I don't understand your objections about scrolling up and down, but maybe I've got my directions reversed. It should definitely look just like you are just scrolling through sequential content; I'm not proposing anything reversed.

(Talking about scrolling up and down is inherently ambiguous because scrollbars, mousewheels, and touchpads all interpret up and down differently.)

[shaneporter]

I wasn't objecting per se. I didn't fully comprehend the intention.

[gordonwoodhull]

I haven't tried it, but it looks like Infinity.js allows variable-height items, as long as the height is not set by CSS (?) and as long as the height doesn't change.

It is also slow to remove items. We currently allow removing error messages but maybe we don't need it once we have #1734 clearing the session pane.

Anyway, this may introduce instability, so it's probably better to attack it in 1.7.

[shaneporter]

That looks good.

[gordonwoodhull]

Confirmed that the session pane will get laggy after "only" a million short rows (with rcloud.session.log(1:1000000)), and will stop printing before another million.