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)

Product:
Core
Component:
Layout: Form Controls
Platform:
x86
Windows NT
Type:
defect

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: buster, Assigned: buster)

References

(
URL
)

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 ago → 24 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.