Closed Bug 278453 Opened 20 years ago Closed 18 years ago

Can't save Signature Placement preference: reverts to default on multiple identities

Categories

(Thunderbird :: Account Manager, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Thunderbird 3

People

(Reporter: chris, Assigned: rob)

References

Details

Attachments

(1 file, 2 obsolete files)

I use Thunderbird with an IMAP email account. I have several identities setup in addition to the primary identity. I set the composition and signature settings for my reply and signature to both be above the quote. It remembers this setting on the primary identity, however on the additional identities it only remembers the reply position. I select to put the signature above and test but it still puts it below the quote. I go back into the settings and it has reverted back to the default below quote setting.
I just want to pile on here. I saw the exact same behavior on Thunderbird 20041206. Any chance of getting this bug confirmed so that it can get fixed?
Is this related to bug 277187 ?
Not that I can tell. It's not that the box is disabled, but that when you change the position that it's not being saved correctly. It only happens on the alternate identities. The primary identity works just fine.
Second that. Exact same behavior on latest release. Let's confirm this sucker and get it fixed.
I confirm this behaviour. However directly in the jprefs.js file editing the entry user_pref("mail.identity.id4.sig_bottom", false); into false instead of true results in the correct behaviour. Apperently the update via the GUI does not work.
(In reply to comment #3) > Not that I can tell. It's not that the box is disabled, but that when you > change the position that it's not being saved correctly. It only happens on > the alternate identities. The primary identity works just fine. I am seeing the exact opposite -- *only* the primary identity does not save that particular setting, as noted at bug 284058 comment 8; and the problem occurs only via the Manage Identities dialog -- if the change is made on the Account Settings' Composition Settings page, it sticks. I see this with TB 1.0.2 and TB 1.0+0414, Win2K. Can you please retest?
*** Bug 291282 has been marked as a duplicate of this bug. ***
The only place that I saw the problem was in the Manage Identities dialog. I retested in 1.0.2 (20050322) and saw the exact same problem. I'm running on Linux, not Windows.
*** Bug 278293 has been marked as a duplicate of this bug. ***
Confirming this bug based on repeated reports, and updating the platform/OS based on comment 8. As noted earlier, I cannot reproduce this exact symptom. Anyone who is encountering this bug, please state whether it's also taking place with TB 1.1a or other current nightlies.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → All
Hardware: PC → All
I am using TB 1.0.6 on Windows XP SP2 on a Dell Inspiron 510m running a Pentium M processor 1.5Ghz and encounter the same problem where the secondary identities do not save the settings for "place signature below my reply". The following are the steps I took to reproduce the bug on the system. Go to Tools -> Account Settings -> Click on Manage Identities (doesnt matter which account) If no additional identities exist, Add a new identity Else, edit the ADDITIONAL identity only (in simple english, the email address which is NOT the default) In the Identity Settings dialog, select the Composition and Addressing Tab. Select the checkbox Automatically quote the original message when replying. Then select the dropdown "start my reply above the quote" Then select the dropdown ... and place my signature "below my reply (above the quote)" Select the OK button. Go to a message in the inbox and REPLY to a message WITH the additional identity as the sender. You will find that the signature is below the quote instead of above the quote. My signature is a small html file conforming to snippets of W3C recommended markup XHTML 1.0
I can confirm this bug. I also confirm Comment #5: editing prefs.js manually works. After that, opening the GUI-config changes it back according to the bug again. One addition: Within the GUI-Account-Settings-editing, after I Edit the signature-location for Identity foo@example.com, and click "OK", I go back to "Identities for example.com". Then I can't click on foo@example.com anymore, it won't turn "blue" as being selected. (OS: Gentoo Linux, tested both TB 1.0.6 and the latest nightly build)
I second this bug. I'm on TB 1.0.2 (20050317) for XP SP2 and those darn sigs keep dropping below the reply! Editing prefs.js works.
Solvation: Comment #5 is correct BUT just remember..... Windows: C:\Documents and Settings\[USERNAME]\Application Data\Thunderbird\Profiles\[SOMENUMBER].default Open: prefs.js IF you have several accounts then remember to edit: (Standard is: "true") user_pref("mail.identity.id1.sig_bottom", false); user_pref("mail.identity.id2.sig_bottom", false); user_pref("mail.identity.id3.sig_bottom", false); user_pref("mail.identity.id4.sig_bottom", false); etc.....
Thanks, that solves it!
I can confirm this behaviour in 1.5 as well. Tried the .js fix and it worked, but would be nice if the GUI would do that though.
*** Bug 324374 has been marked as a duplicate of this bug. ***
Im running TB 1.5 (20051201) and i can confirm that the .js fix, fix the problem, however when changing identities inside the composition windue it does add the new marked signature, but it forgets to remove the other signatur ... :(
(In reply to comment #18) > Im running TB 1.5 (20051201) and i can confirm that the .js fix, fix the > problem, however when changing identities inside the composition windue it > does add the new marked signature, but it forgets to remove the other > signatur ... That's bug 218346.
*** Bug 344123 has been marked as a duplicate of this bug. ***
There's been a patch checked in (11 Jul) for bug 284058, which is similar to but distinct from this bug. Scott said he couldn't see the fix working for that bug, but it seemed to fix it for me. Could anyone who's experienced this problem check with 2a1 or a trunk build to see if it's still occurring?
I have this same issue and can confirm that in thunderbird-3.0a1.en-US.linux-i686.tar.bz2 it still exists.
*** Bug 353653 has been marked as a duplicate of this bug. ***
updating summary for searchability
Summary: Signature Preference reverts to default on multiple identities → Can't save Signature Placement preference: reverts to default on multiple identities
Still existing in Windows Thunderbird Beta 2
We are implicitly casting a string to boolean when copying the control value back to the identity object during save. This doesn't work because any non-empty string evaluates to true. So I added an explicit conversion to the proper boolean value.
Attachment #258563 - Flags: review?
Attachment #258563 - Flags: review? → review?(mscott)
Attachment #258563 - Flags: superreview?(neil)
Maybe it's because i am new. but how do i install this patch?
Comment on attachment 258563 [details] [diff] [review] Patch to save signature placement in secondary identities >+ identity.sigBottom = (document.getElementById('identity.sig_bottom').value == 'true') ? true : false; The ?: is unnecessary because == already returns a boolean.
Attachment #258563 - Flags: superreview?(neil) → superreview+
Comment on attachment 258563 [details] [diff] [review] Patch to save signature placement in secondary identities r=mscott if you address Neil's comment
Attachment #258563 - Flags: review?(mscott) → review+
Attachment #258563 - Attachment is obsolete: true
Attached patch Revised patch for saving signature placement (obsolete) (deleted) — Splinter Review
Removed unnecessary code per review comments. Please review and apply at your convenience.
Attachment #258671 - Flags: superreview?(neil)
Attachment #258671 - Flags: review?(mscott)
Comment on attachment 258671 [details] [diff] [review] Revised patch for saving signature placement You can remove the parens too. I don't need to re-review it. Thanks for the patch!
Attachment #258671 - Flags: review?(mscott) → review+
Comment on attachment 258671 [details] [diff] [review] Revised patch for saving signature placement And whomever removes the parens can remove that blank line I overlooked last time too, and that definitely doesn't need rereview, so when you're done, just put [checkin needed] on the whiteboard and one of the Mozilla interns should spot that and check it in for you.
Attachment #258671 - Flags: superreview?(neil) → superreview+
Attached patch Changed formatting (deleted) — Splinter Review
Changed formatting.
Attachment #258671 - Attachment is obsolete: true
(In reply to comment #34) > just put [checkin needed] on the whiteboard and one of the Mozilla interns should > spot that and check it in for you. Hmmm... how do I set the whiteboard field? Don't seem to have access. (Excuse my ignornace. This is my first patch to Mozilla.)
Thanks so much for the patch Rob!
Whiteboard: [checkin needed]
Assignee: mscott → rhasselbaum
Checked in, thanks Rob. mozilla/mailnews/base/prefs/resources/content/am-identity-edit.js 1.11 > ...and one of the Mozilla interns Or someone else, you know.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
Thank you, I've fixed this manually by changing .js file, but I think this is must fixed to allow user to change this option via the GUI, am I right?
As I understand it, the fix was checked into the trunk, but not the branch used for TB 2.0. So it is not in 2.0.
(In reply to comment #14) > Solvation: > Comment #5 is correct BUT just remember..... > > Windows: > C:\Documents and Settings\[USERNAME]\Application > Data\Thunderbird\Profiles\[SOMENUMBER].default > > Open: prefs.js > > IF you have several accounts then remember to edit: > (Standard is: "true") > user_pref("mail.identity.id1.sig_bottom", false); > user_pref("mail.identity.id2.sig_bottom", false); > user_pref("mail.identity.id3.sig_bottom", false); > user_pref("mail.identity.id4.sig_bottom", false); > etc..... > Nikolaj, I was on version 2.0.0.6 (20070728) and this silly bug has still not been fixed!? I followed your method above and it worked beautifully for me. Finally this has been solved. Thanks!
QA Contact: account-manager
Target Milestone: --- → Thunderbird 3
See Bug 441242 for additional info.
This is confirmation that the issue has been resolved in Vista Home Premium, Shredder version 3.0a2pre (2008071503). Thank you.
Thank you for resolution of this bug!
Status: RESOLVED → VERIFIED
Please, reopen this bug for Shredder 3.0b3pre (20090327) I have: - 2 identities, one with signature, other without - signatures to start above quoted text - place signature above quoted text (below reply) When replying, the signature doesn't go away when switching to tne 'no signature' identity and I end up with 2 signatures stacked if I change back and they keep stacking if I keep changing.
Flags: wanted-thunderbird3?
(In reply to comment #57) Your issue seems like it is somewhat different. I am using TB trunk, and not seeing any issue at all at all. So maybe open a new bug.
Flags: wanted-thunderbird3?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: