Closed Bug 29381 Opened 20 years ago Closed 17 years ago

CVS Blame and diffs have gaps in 5.0

Categories

(Webtools Graveyard :: Bonsai, defect, P3)

x86
Linux
defect

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: dbaron, Assigned: endico)

References

()

Details

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>
This also applies to the CVS diffs (although they have other problems caused by
the unterminated PRE elements as well).
Summary: CVS Blame has gaps in 5.0 → CVS Blame and diffs have gaps in 5.0
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
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...)
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.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → WORKSFORME
QA Contact: lxr
Component: LXR → Bonsai
QA Contact: lxr → bonsai
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.