Closed Bug 3172 Opened 26 years ago Closed 26 years ago

{feature} pre wrap cols=x doesn't wrap

Categories

(Core :: Layout, defect, P2)

x86
Linux
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: akkzilla, Assigned: buster)

References

()

Details

See the cited URL, which uses <pre wrap width=68>. The long line should wrap, and does on 4.5, but doesn't on seamonkey (on either Linux or Windows). Mail/news people will need this for compose window dogfood (plaintext editor).
QA Contact: 4110 → 4082
Reassigning QA contact to glynn@netscape.com (assigned to PRE element).
Assignee: troy → kipp
Summary: pre wrap width=68 doesn't wrap → pre wrap cols=x doesn't wrap
Okay, an update. It turns out that "cols" rather than "width" is actually the approved attribute, and we have two bugs: 1. "cols" doesn't work. I've modified the test page pre.html to wrap at 10 cols. This works on 4.5, but not on viewer. This functionality will be important for plaintext editing. 2. "wrap" doesn't. On 4.5, wrap with no cols wraps to the window size; on 5.0 it doesn't wrap at all, we get a horizontal scrollbar. This is less important for plaintext editing milestone; we'll probably be specifying column width explicitly for the first pass. There's also a question of whether "width" should have any effect inside a pre wrap tag; I'm not sure of the answer to that, emacs' html-mode thinks width is a valid modifier to pre and I thought I saw another reference which agreed, but can't find it now.
Cc:ing self. kipp, is there any possibility of getting this done for M4, so that we can get plain text mail compose working better?
pre width is deprecated and a bug has been filed concerning it not working for backwards compat. I do not see a pre wrap or cols attribute in HTML 4.0 or 3.2 spec (?).
Severity: normal → enhancement
Priority: P2 → P5
Summary: pre wrap cols=x doesn't wrap → {feature} pre wrap cols=x doesn't wrap
Severity: enhancement → critical
Priority: P5 → P2
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
I just checked in the implementation for this; note that its not 100% compatible with nav4...
Status: RESOLVED → VERIFIED
VERIFIED Fixed for Linux 1999060112 and WinNT 1999060108.
You need to log in before you can comment on or make changes to this bug.