Closed
Bug 3172
Opened 26 years ago
Closed 26 years ago
{feature} pre wrap cols=x doesn't wrap
Categories
(Core :: Layout, defect, P2)
Tracking
()
VERIFIED
FIXED
M6
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).
Updated•26 years ago
|
QA Contact: 4110 → 4082
Comment 1•26 years ago
|
||
Reassigning QA contact to glynn@netscape.com (assigned to PRE element).
Reporter | ||
Updated•26 years ago
|
Summary: pre wrap width=68 doesn't wrap → pre wrap cols=x doesn't wrap
Reporter | ||
Comment 2•26 years ago
|
||
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.
Comment 3•26 years ago
|
||
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 (?).
Summary: pre wrap cols=x doesn't wrap → {feature} pre wrap cols=x doesn't wrap
I just checked in the implementation for this; note that its not 100% compatible
with nav4...
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Comment 6•26 years ago
|
||
VERIFIED Fixed for Linux 1999060112 and WinNT 1999060108.
You need to log in
before you can comment on or make changes to this bug.
Description
•