29381 - CVS Blame and diffs have gaps in 5.0

Status: RESOLVED WORKSFORME

(Webtools Graveyard :: Bonsai, defect, P3)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

CVS Blame (see the CVS blame on any file, for example, the URL above) has lots
of gaps in 5.0 because it exploits a bug in 4.x to prevent margins from showing
on PRE elements - 4.x doesn't put margins around unterminated pre elements.

The correct way to avoid margins on pre elements is to include a style rule that
suggests that PRE elements should not have margins.  I think including the
following in the HEAD of the files served should fix the problem:

<style type="text/css">
pre { margin: 0; padding: 0; }
</style>

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

This also applies to the CVS diffs (although they have other problems caused by
the unterminated PRE elements as well).

Comment 2

[Gervase Markham [:gerv]]

This bug has not been touched for more than nine months. In most cases, that 
means it has "slipped through the net". Please could the owner take a moment to 
add a comment to the bug with current status, and/or close it.

Thank you :-)

Gerv

Comment 3

[Andreas Kunz]

dbaron, well, I think this lxr bug is fixed now, isn't it?
(At least I cannot see it and it is almost 3 years old...)

Comment 4

[Andreas Kunz]

Ok, I won't let this bug get 3 years old (and it is only a week from that).
This has been working for months and probably years.
-> WFM

Please reopen if I'm a complete idiot and this is still a bug.