Code review comment for lp:~thumper/launchpad/code-review-comment-formatting

Revision history for this message
Tim Penhey (thumper) wrote :

= Summary =

Fixes bug 326129 (firefox-3.{5,6} bug, merge review comments don't get wrapped
in <pre> block).

== Proposed fix ==

Adding the "white-space: pre-wrap" to the nice_pre formatted <pre> tag.

== Implementation details ==

Uses the "pre.wrap" CSS style now, rather than inline styles.

I was reminded of why we don't use the bug formatter for code review comments.
The bug formatter is turning an email into HTML, by grouping lines into
paragraphs. This works really well for actual conversations, but is complete
shite for code.

A question for beuno:
  should the pre.wrap class be added to style-3-0.css?

== Demo and Q/A ==

Look at a code review comment :)

== Reviewers ==

Need a general review:
  reviewer launchpad
And a UI review
  reviewer beuno ui
And a QA review to actually test against FF3.5, Konq, Opera and IE
  reviewer launchpad browser-qa

« Back to merge proposal