1357920 - Port bug 1354002 - Set execCommand("defaultparagraphseparator", false, "br/p") depending on mail.compose.default_to_paragraph to SeaMonkey

Status: RESOLVED FIXED

Description

[Ian Neal]

Comment 1

[Ian Neal]

Created attachment 8859774 [details] [diff] [review]
Add mail.compose.default_to_paragraph check [Checked in: Comment 2]

Simple port of TB patch

Comment 2

[Ian Neal]

Comment on attachment 8859774 [details] [diff] [review]
Add mail.compose.default_to_paragraph check [Checked in: Comment 2]

https://hg.mozilla.org/comm-central/rev/d67289948650

Comment 3

[Jorg K (GMT+2)]

Sorry, I had to back this out since it's causing unwanted side-effects in replies, see bug 1354002 comment #11. I've also backed out that bug.
https://hg.mozilla.org/comm-central/rev/38b43436a7c0fcb28c3d260091f1572646ba3c61

Comment 4

[Jorg K (GMT+2)]

I've done some more work in this in bug 1354002, including some mailnews changes. So this got now relanded, but with an additional hunk since in paragraph mode mailnews now no longer produces "br"s.

Can someone please check that all is well in SM.

https://hg.mozilla.org/comm-central/rev/064ea09a484db674bea276f4f634dd9b7bf85d24

I've also moved the target forward to SM 2.53 which equates TB 56.

Comment 5

[Frank-Rainer Grahl (:frg)]

I am now testing this for a while and the current behaviour it to swallow blank lines at the end of a parapraph. Not reopening it. Can't tell if this patch or something else caused it and no time to nail it down right now.

Comment 6

[Jorg K (GMT+2)]

I'm happy with the behaviour of TB after bug 1354002 got landed:
https://hg.mozilla.org/comm-central/rev/dd97865d7a80a00490e1cf1226b05ca6d2fe18bc
I haven't seen any bad effects.

What do you mean by "swallow blank lines at the end of a paragraph"?

Comment 7

[Frank-Rainer Grahl (:frg)]

I am seeing this mostly in newsgroup text only posts. It seems blank lines get removed where a br in html posts should/would be inserted. Just an uneducated guess right now because I am not that firm with this code and prefs. I am filing it under issue 999 right now until I find more time to check my settings and analyse the raw data. 

Just check my latest post in mozilla.dev.apps.seamonkey as an example.

> Re: l10n.mozilla-community.org?

> https://bugzilla.mozilla.org/show_bug.cgi?id=1383642
> FRG

Missing the blank line between the link and frg which was there during compose.

Comment 8

[Jorg K (GMT+2)]

Hmm, I tried a few scenarios and didn't lose any blank line.

Can you reproduce the problem? If so, can you save the message as a draft and attach it here?
Also make sure that editing the draft and then sending it will reproduce the problem.