[TRAC] prefs box overlaps long lines in message text

[jstressman]

On trac messages, if you have a very long line of text, the floating preferences box on the right side overlaps the message text due to the use of a 'pre' tag for the message text causing the message text not to wrap.

Adding the following line to ​http://dev.haiku-os.org/chrome/common/css/diff.css in the rule "#overview dd.message pre" will fix the problem in modern browsers:

white-space: pre-wrap;

This allows the diff.css file to still validate as valid CSS 2.1 with the W3 CSS validator.

(I can send this upstream to the official Trac people if need be?)

[jstressman]

I wanted to add that while this allows the 'pre' wrapped text to wrap on the page, it doesn't affect the text itself, so copy/pastes of the message itself will still retain the original format, line breaks, etc. So it shouldn't have any negative effect on code snippets etc.

[nielx]

Do you have an example ticket?

[Disreali]

This is still an issue when reading changeset pages. If I understood the description correctly, the attached patch should help.

[jstressman]

This is still present, and the patch it still the same. I've just tested both again.

Do I need to send this upstream? (or has it been already?)

[nielx]

Replying to jstressman:

This is still present, and the patch it still the same. I've just tested both again.

Do I need to send this upstream? (or has it been already?)

Yes, please push it upstream. A ticket search did not yield any results, but please verify (I searched for all components in version control wit the description search word 'wrap').

[Disreali]

Was the patch ever upstreamed?

[zooey]

I have no idea if the patch was ever upstreamed, but I have applied it to our own Trac buildservice project and -customizations repo. The patch indeed fixes the wrapping of messages - thanks!

[Disreali]

Thank you. It works well in Web+ but not Bezilla. I think our Bezilla is just to old at this point.

[edited to fix typo]