458475 - Diff lines overlapping in long lines

Status: VERIFIED WORKSFORME

(Bugzilla :: Attachments & Requests, defect)

Description

[Tyler Downer [He/Him]]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1b1pre) Gecko/20081003 Minefield/3.1b1pre
Build Identifier: 

I will attach a screenshot later. When this diff is viewed, several lines overlap from the original column to the new column. Specifically, some of these exhibiting this are lines 56 and 57 on the top file (it happens in several other places). Zoom is at normal. It seems that the line endings are longer that the viewable area, so it overlaps. This is most likely not the best solution. Maybe breaking it down to the next line, but with a line continuation mark.

Reproducible: Always

Steps to Reproduce:
1.Visit given URL
2.View Lines 56, 57

Actual Results:  
Lines overlap next column. 

Expected Results:  
No overlap.

Comment 1

[Tyler Downer [He/Him]]

Attachment: Screenshot of the line overlap

Comment 2

[Frédéric Buclin]

There is something wrong with your Fx profile. Lines should correctly wrap, at least in Firefox. Visiting the link above, I can see long lines being wrapped, in Fx 3.0.3 and Fx 3.1b1pre. But it indeed doesn't wrap correctly using Opera 9.52.

Comment 3

[Tyler Downer [He/Him]]

OK, I will check with a new profile, though this one is fresh, so I don't see that, but will double-check.

Comment 4

[Tyler Downer [He/Him]]

I tried in a new profile with no changes. The overlapping issue is still there. Do you think this is a FF bug? I am using Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1b1pre) Gecko/20081004 Minefield/3.1b1pre FYI.

Comment 5

[Max Kanat-Alexander]

Yeah, I can't reproduce it either.

Comment 6

[Frédéric Buclin]

(In reply to comment #4)
> Do you think this is a FF bug?

Doubtful, unless it's specific to Firefox on Windows. But as I said, I can reproduce with Opera, so reopening.

Comment 7

[Tyler Downer [He/Him]]

Now that bmo has been upgraded, to 3.2RC2, I am not seeing this issue. I will close for now.

Comment 8

[Tyler Downer [He/Him]]

Not seeing anymore.