I don't think this is the approriate fix.

The git example is one area where this happens; I'll be there's a few others, and 99% of the time code never overruns the box. When it does I think having a scrollable box is way better than lines running off into the sidebar!

IMO, those places should be fixed individually by following good practices, not by adding a scrollbar (which is really not a good usability solution).

I won't approve this PR but I am not blocking it either.

You should use auto instead of scroll so that only the few boxes that need it get a scrollbar.

I agree that we should try as much as possible to get rid of overly wide boxes, but 1. it's not so easy to check them (AFAICT) and 2. you never know what the window width of the reader is.

thanks!

backported to v2.0.0-doc as fa76580