fixes #2991 by adjusting the y scroll after max_lines exceeded

[mardanst]

Please review the following checklist.

• Docstrings on all new or modified functions / classes
• Updated documentation
• Updated CHANGELOG.md (where appropriate)

[rodrigogiraoserrao]

Hey @mardanst, thanks for the PR.

Before we take a look at it, could you make sure the tests pass locally?
You can go through the steps outlined in #2923, from the section "Guidelines" onwards.
Thanks!

[mardanst]

Thanks @rodrigogiraoserrao, that link was very helpful. I was able to fix the failing test and add a new snapshot test for #2991 .

Tests are passing locally, but failing in github on (windows-latest,3.11) on an unrelated test.

[mardanst]

Resubmitted it and it passed the second time through

[rodrigogiraoserrao]

Thanks for following up on it. We'll take a look and get back to you.

As you realised, some of our tests are a bit flakey. When one of those fails, no need to create a "fake" commit to trigger them again. We'll rerun them for you :)

[rodrigogiraoserrao]

Hey @mardanst, sadly we let this slip and now there have been some modifications to TextLog (which is now RichLog) and we've added a new Log widget. (See #3046 for some more context.)

Are you keen on bringing your PR up to date (possibly closing this one and making a new one against the head of main), or do you want to dissociate yourself from this PR and leave it up to us to fix the issue that this PR tackles?