Fix word-wrap styling of <pre> tags in status updates.

Review Request #14057 — Created July 26, 2024 and submitted — Latest diff uploaded

Information

Review Board
release-7.x

Reviewers

The reviews CSS generally has the same rules apply to regular reviews
and reviews included in status updates, but one thing that was missing
was the word-wrap rule applied to <pre> tags. This was buried inside
of #reviews .review, which isn't used for the status update entries.

This change consolidates everything under a single rule for
pre.reviewtext, which covers all comments and review body fields
across regular reviews and status updates.

Created a review request with lots of different reviews and comments,
as well as initial and update status updates, which included long
single-line text for all the various fields. Saw that everything wrapped
correctly.

Diff Revision 1 (Latest)

Commits

First Last Summary ID Author
Fix word-wrap styling of <pre> tags in status updates.
The reviews CSS generally has the same rules apply to regular reviews and reviews included in status updates, but one thing that was missing was the `word-wrap` rule applied to `<pre>` tags. This was buried inside of `#reviews .review`, which isn't used for the status update entries. This change consolidates everything under a single rule for `pre.reviewtext`, which covers all comments and review body fields across regular reviews and status updates. Testing Done: Created a review request with lots of different reviews and comments, as well as initial and update status updates, which included long single-line text for all the various fields. Saw that everything wrapped correctly.
733808ccadd34239d0f3a78f20513be6698035fc David Trowbridge
reviewboard/static/rb/css/pages/reviews.less
Loading...