Closed
Bug 1385725
Opened 7 years ago
Closed 7 years ago
Port bug 1336531 to SM: <br>-Tag before signatures
Categories
(SeaMonkey :: Composer, defect)
SeaMonkey
Composer
Tracking
(seamonkey2.48 wontfix, seamonkey2.49esr fixed, seamonkey2.50 wontfix, seamonkey2.51 wontfix, seamonkey2.52 wontfix, seamonkey2.54 fixed, seamonkey2.53 fixed)
RESOLVED
FIXED
seamonkey2.54
People
(Reporter: jorgk-bmo, Assigned: jorgk-bmo)
References
Details
Attachments
(1 file)
4.46 KB,
patch
|
frg
:
review+
iannbugzilla
:
approval-comm-release+
iannbugzilla
:
approval-comm-esr52+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1336531 +++ In bug 1336531 I changed composition to not include an additional <p> before a signature. The necessary mailnews changes were done with |#ifdef MOZ_THUNDERBIRD|. We should eventually remove that and port the changes to SM. I think all you need are these two hunks: https://hg.mozilla.org/comm-central/rev/a25600ed9d2e#l1.78 https://hg.mozilla.org/comm-central/rev/a25600ed9d2e#l1.96 With those in place, we can lose: https://hg.mozilla.org/comm-central/rev/a25600ed9d2e#l3.12 https://hg.mozilla.org/comm-central/rev/a25600ed9d2e#l3.24 https://hg.mozilla.org/comm-central/rev/a25600ed9d2e#l3.46
Assignee | ||
Comment 1•7 years ago
|
||
Here is what I think you should do. I won't compile or test it, so please take the necessary action.
Flags: needinfo?(frgrahl)
Comment 2•7 years ago
|
||
Thanks jorg, I will try it out. I was just able to reproduce my problem with the swallowed blanks and it has nothing to do with signatures. Can reproduce it 100% with newsgroup replies and in TB Daily too :) Stay tuned for a new bug but not today.
Assignee | ||
Comment 3•7 years ago
|
||
(In reply to Frank-Rainer Grahl (:frg) from comment #2) > Stay tuned for a new bug but not today. I can't wait to see it ;-)
Assignee | ||
Comment 4•7 years ago
|
||
Comment on attachment 8891820 [details] [diff] [review] 1385725-sm.patch No action here?
Attachment #8891820 -
Flags: review?(frgrahl)
Comment 5•7 years ago
|
||
Sorry still had NI set to look into it but something always came up. Will see that I put it on top of my list.
Comment 6•7 years ago
|
||
Comment on attachment 8891820 [details] [diff] [review] 1385725-sm.patch Works great thanks. I was able to confirm the wrong behaviour in SeaMonkey. Looking at the generated eml messages before and after the patch I can now see that the <br> is gone. Let me know if you want to check it in yourself. [Approval Request Comment] Regression caused by (bug #): -- User impact if declined: Visual glitch in output message Testing completed (on m-c, etc.): c-r Risk to taking this patch (and alternatives if risky): none already in TB String changes made by this patch: --
Flags: needinfo?(frgrahl)
Attachment #8891820 -
Flags: review?(frgrahl)
Attachment #8891820 -
Flags: review+
Attachment #8891820 -
Flags: approval-comm-release?
Attachment #8891820 -
Flags: approval-comm-esr52?
Attachment #8891820 -
Flags: approval-comm-beta?
Updated•7 years ago
|
Assignee: nobody → jorgk
Status: NEW → ASSIGNED
Updated•7 years ago
|
status-seamonkey2.48:
--- → wontfix
status-seamonkey2.49esr:
--- → affected
status-seamonkey2.50:
--- → wontfix
status-seamonkey2.51:
--- → wontfix
status-seamonkey2.52:
--- → wontfix
status-seamonkey2.53:
--- → affected
status-seamonkey2.54:
--- → affected
Assignee | ||
Comment 7•7 years ago
|
||
https://hg.mozilla.org/comm-central/rev/0c4221f192584abffbb1d674eb309f90db25898f Port bug 1336531 to SM: No <br>-Tag before signatures in paragraph mode. r=frg Yes, I wanted to check it in myself. I assume you handle the approvals and uplifts. Due to the mailnews hunk, uplifts will trigger builds, so use DONTBUILD where appropriate.
Assignee | ||
Comment 8•7 years ago
|
||
Never got closed.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.54
Comment 9•7 years ago
|
||
Upps and should probably be put into the next 2.49 too. IanN could you do the a+ for 2.53 and 2.49 if ok.
Flags: needinfo?(iann_bugzilla)
Updated•7 years ago
|
Attachment #8891820 -
Flags: approval-comm-beta?
Comment 10•7 years ago
|
||
Comment on attachment 8891820 [details] [diff] [review] 1385725-sm.patch a=me
Flags: needinfo?(iann_bugzilla)
Attachment #8891820 -
Flags: approval-comm-release?
Attachment #8891820 -
Flags: approval-comm-release+
Attachment #8891820 -
Flags: approval-comm-esr52?
Attachment #8891820 -
Flags: approval-comm-esr52+
Comment 11•7 years ago
|
||
Fixed for 2.49.2 https://hg.mozilla.org/releases/comm-release/rev/b1a70a8dd3684cb9300dfda1a5fb15f6cc408896 https://hg.mozilla.org/releases/comm-esr52/rev/a00f14e4cac1021d7186c624ebeecc66d29f71dc
You need to log in
before you can comment on or make changes to this bug.
Description
•