Closed Bug 450342 Opened 12 years ago Closed 11 years ago

Diff view overlaps the sidebar content

Categories

(support.mozilla.org :: General, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: stephend, Assigned: ecooper)

References

()

Details

(Keywords: ue, Whiteboard: sumo_only theme)

Attachments

(3 files)

Assignee: nobody → smirkingsisyphus
Severity: major → normal
Target Milestone: --- → 0.8
Ugh. Word wrapping...The solution to this will probably be the same as Bug 413579.

Aside from just clipping the output using overflow, there aren't many options available for CSS, We could use word-break (in CSS3, I believe), but it doesn't work in Firefox 3.

<wbr>'s don't validate and don't work with Opera, so inserting some &#8203;'s is probably the better solution. I think they work on all modern browsers.
I have no real opinion on the implementation, but your proposal sounds reasonable. Is there a way we could test/demo how that would work?
&#8203; and &shy; are both zero-width spaces. So, you throw some in a string every some many characters (every other, every five, whatever), the line will break if it starts to overflow.

http://www.quirksmode.org/oddsandends/wbr.html

The above shows examples of <wbr>, &#8203;, and &shy; in action. When I implement it, you could just view it on my dev server first, which is accessible via the web most times.
Looks like &shy; is the best option, no? Either way, the proposal sounds like a good solution!
Target Milestone: 0.8 → 0.9
Target Milestone: 0.9 → 1.1
Eric, what's the status of this? If you have anything on this but are too busy, I can take it up and implement it. There are a bunch of bugs on word-breaking that'd be good to fix for 1.1
Priority: -- → P3
Blocks: 440583
Target Milestone: 1.1 → 1.0
Target Milestone: 1.0 → 1.0.2
Attached image Broken diffs?
Are diffs broken in general?

I was testing a fix before I realized they look fundamentally different than the original sreenshot now.

Loading the link in the bug summary will display you a diff that looks like the one in this screenshot.
Yeah, that looks broken to me :-(
The first screenshot is comparing versions 11 and 12. The second screenshot is comparing versions 10 and 12.
Attached patch v1Splinter Review
This is combination of adding a zero-width break and some CSS rules.

The regex is used because wordwrap kills entities.
Attachment #372767 - Flags: review?(laura)
Attachment #372767 - Flags: review?(laura) → review+
r24453/r24454

Aside from making sure diffs stay within bounds, we should be checking that all side-by-side diffs still work.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: sumo_only theme
You need to log in before you can comment on or make changes to this bug.