Closed Bug 21032 Opened 25 years ago Closed 24 years ago

[regression] can paste multiple lines of text into a single-line text control

Categories

(Core :: Layout: Form Controls, defect, P3)

x86
Windows NT
defect

Tracking

()

RESOLVED INVALID

People

(Reporter: buster, Assigned: buster)

References

()

Details

until recently, you could only paste a single line of text into an <input
type=text>.  sometime in the recent past, this broke, and you can paste multiple
lines of text.
Target Milestone: M13
verified this on the 19991207 build on win32
Whiteboard: fix in hand
the problem was that the InsertText code no longer uses InsertBreak to insert
<BR>s.  Instead, we just put the newlines into the content model directly.
That's fine, but we were relying on InsertBreak() to check against mFlags to see
if we were a single line text control or not, and if so it would skip the
insertion of the <BR>.  Now, InsertText() needs to remove newlines itself.
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Whiteboard: fix in hand
fixed by replacing CRLF with spaces
*** Bug 22388 has been marked as a duplicate of this bug. ***
I think this bug has regressed again and needs reopening...
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
i have memories of someone removing the crlf conversion.  It was either me or 
someone I reviewed.  1.5 weeks of Florida vacation have erased further details 
from my mind.  I'll look into it. reopening bug...
ok, I talked to akkana about this.  This effect is intentional.  See bugs 23485 
and 50935.  closing bug invalid.
Status: REOPENED → RESOLVED
Closed: 25 years ago24 years ago
Resolution: --- → INVALID
Updating QA contact.
QA Contact: ckritzer → bsharma
*** Bug 22388 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.