remove horizontal scroll bars on code blocks

[ludde]

Don't display diffs in a scrollable textbox (or IFRAME or whatever it is).

For large diffs, the horizontal scrollbar is not visible, so you need to scroll down to the very end of the document to be able to scroll to the right.

A better solution is to put the diff in-line in the page and make use of the browser's horizontal scrollbar.

[Matthew Good]

The scrollable blocks are just <div> or <pre> blocks with the CSS style rule overflow: auto, which tells the browser to add scroll bars to the block if the content exceeds the block's width.

These rules were added to prevent the content of a code block from spilling out into the surrounding content which looked sloppy. If you prefer the content to spill out of the block see TracInterfaceCustomization for overriding the CSS rules.

[ludde]

Maybe a different solution should be investigated then. It's doesn't really follow any usability guidelines to force a user to scroll down to the end of a page before the horizontal scrollbar is visible.

This is imho a major interface flaw that needs to be rectified.

[Matthew Good]

After discussion with ludde on the IrcChannel, this is generally a problem with file displays for either the repository browser and attachments. Embedded blocks should generally be ok as they are.

For the file display, we could handle it similarly to the display in the changeset view. Annotate the files with line numbers to the left and allow the text to line-wrap in order to prevent the need for horizontal scrolling.

[Matthew Good]

Add support to default mimeviewer to display text in table with line numbers

[Matthew Good]

I've attached a partial patch for supporting the line numbers in the default mimeviewer. I will try to add support for at least the Silvercity mimeviewer before checking this in.

[Matthew Good]

The fix for #520 in [1764] also fixes this ticket.