310201 - D&D: dropped text is double-spaced

Status: RESOLVED FIXED

(Core Graveyard :: Widget: OS/2, defect)

Description

[Rich Walsh]

While responding to Bug #267609, I dragged some text from a file & dropped
it into the Comments text area.  Although it was correctly formatted when
I dropped it, it was double-spaced after I committed the comment.

My guess is that the CrLf is being interpreted as two separate linebreaks.
This patch should fix the problem by eliminating all carriage returns from
dropped text.  The text you're reading will confirm whether my guess is
correct because it was composed with explicit CrLf linebreaks and was
dropped into the Comments text area.

Comment 1

[Rich Walsh]

Attachment: remove CRs from dropped text

looks like it works...

Comment 2

[Mike Kaply [:mkaply]]

Comment on attachment 197593 [details] [diff] [review]
remove CRs from dropped text

Seems like a lot of work to remove carriage returns... :)

Comment 3

[Rich Walsh]

(In reply to comment #2)
> Seems like a lot of work to remove carriage returns... :)

I can't say that I understand this comment.  Allocating a new buffer for the
results would require more work, as would examining the text character by character.

Comment 4

[Mike Kaply [:mkaply]]

(In reply to comment #3)
> I can't say that I understand this comment.  Allocating a new buffer for the
> results would require more work, as would examining the text character by
character.

True. If only it was an nsString....

Anyway, I'll check this in asap.

Comment 5

[Peter Weilbacher]

From your description it could just as well be a bug in Bugzilla but I just
confirmed the problem with a webmail site. Still, I wonder if there there is a
standard that says that text in HTML forms should only be LF and not CRLF? I
didn't find it but I could just have used stupid search parameters...

The problems is also present on the 1.8 branch.

Comment 6

[Peter Weilbacher]

I found RFC 1867 which says in section 5.9 that CRLFs are to be used as line separators in HTML forms. But forms get converted to CRLF anyway on submission (somewhere in nsFormSubmission.cpp) so that this is indeed the correct fix.

Comment 7

[Mike Kaply [:mkaply]]

Comment on attachment 197593 [details] [diff] [review]
remove CRs from dropped text

OS/2 only fix

Comment 8

[Mike Schroepfer]

Comment on attachment 197593 [details] [diff] [review]
remove CRs from dropped text

OS/2 Only - approved to land