Copies & Folders: Initialize Bcc field to the current address

VERIFIED FIXED in Thunderbird 13.0

Status

MailNews Core
Account Manager
P2
enhancement
VERIFIED FIXED
14 years ago
4 years ago

People

(Reporter: Bruce K Gillespie, Assigned: aceman)

Tracking

Trunk
Thunderbird 13.0
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

14 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4) Gecko/20030529
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4) Gecko/20030529

When selecting bcc in the copies & folders for an account is does not fill in
the address.

Reproducible: Always

Steps to Reproduce:
1. For an account. Select Copies & Folders.
2. Click the Bcc button.


Actual Results:  
Address field is blank.

Expected Results:  
Address field should be filled in with the current accounts address.

This was behaviour in 1.3.1 and before.

Comment 1

14 years ago
*** Bug 221523 has been marked as a duplicate of this bug. ***

Comment 2

14 years ago
The difference that this bug complains about was implemented specifically in the 
fix for bug 178328.  Previously there were three checkboxes for managing Sent 
messages; the two regarding Bcc: were combined into one.

It seems a reasonable request to ask that the text field be initialized to the 
account's default identity, so I'm changing this bug to an enhancement and 
tweaking the summary accordingly.
Severity: normal → enhancement
OS: Windows XP → All
Hardware: PC → All
Summary: Bcc setting in Copies & Folders no longer defaults to the current address → Copies & Folders: Initialize Bcc field to the current address
Product: Browser → Seamonkey

Updated

13 years ago
Assignee: sspitzer → mail
This is an automated message, with ID "auto-resolve01".

This bug has had no comments for a long time. Statistically, we have found that
bug reports that have not been confirmed by a second user after three months are
highly unlikely to be the source of a fix to the code.

While your input is very important to us, our resources are limited and so we
are asking for your help in focussing our efforts. If you can still reproduce
this problem in the latest version of the product (see below for how to obtain a
copy) or, for feature requests, if it's not present in the latest version and
you still believe we should implement it, please visit the URL of this bug
(given at the top of this mail) and add a comment to that effect, giving more
reproduction information if you have it.

If it is not a problem any longer, you need take no action. If this bug is not
changed in any way in the next two weeks, it will be automatically resolved.
Thank you for your help in this matter.

The latest beta releases can be obtained from:
Firefox:     http://www.mozilla.org/projects/firefox/
Thunderbird: http://www.mozilla.org/products/thunderbird/releases/1.5beta1.html
Seamonkey:   http://www.mozilla.org/projects/seamonkey/
This bug has been automatically resolved after a period of inactivity (see above
comment). If anyone thinks this is incorrect, they should feel free to reopen it.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → EXPIRED

Comment 5

7 years ago
Still valid and reasonable.
Assignee: mail → nobody
Status: RESOLVED → REOPENED
Component: MailNews: Account Configuration → Account Manager
Ever confirmed: true
Product: SeaMonkey → MailNews Core
QA Contact: nbaca → account-manager
Resolution: EXPIRED → ---
Whiteboard: [good first bug]

Updated

7 years ago
Status: REOPENED → NEW
(Assignee)

Comment 6

6 years ago
It seems I could try to patch this.
Assignee: nobody → acelists
Priority: -- → P2
(Assignee)

Comment 7

6 years ago
Created attachment 584094 [details] [diff] [review]
init BCC and CC if selected, clear them if deselected
Attachment #584094 - Flags: ui-review?(bwinton)
(Assignee)

Updated

6 years ago
Status: NEW → ASSIGNED
Comment on attachment 584094 [details] [diff] [review]
init BCC and CC if selected, clear them if deselected

So, I mostly like this, but there is an odd case that I'm not sure about.

If I check "Cc", then delete my email address, then un-check "Cc", then re-check it, I don't think I expect to see my email address appear there again…

It would help mitigate this if, when I clicked "Cc", the text field was focused, and my email address was selected, so that if I didn't want it (or wanted a different address), all I would need to do would be to hit delete (or start typing).

But that's a fairly minor complaint, and fairly minor change to help it, so I'll say ui-r=me with that implemented.

Thanks,
Blake.
Attachment #584094 - Flags: ui-review?(bwinton) → ui-review+
(Assignee)

Comment 9

6 years ago
Created attachment 588780 [details] [diff] [review]
patch v2

Select BCC field if we initialized it. Clear the emails when it is deselected and it is still at out inited value. Do not touch other user-edited value. (The value is only lost if the user moves to other account and selects Copies & Folders there (see further)).
This change made a variant of bug 690277 visible here (it was here even before this patch I think).
Select Copies & Folders in one account, set some values in the BCC textbox. Directly move to Copies & Folders in another account that does not have any values set in the textbox (and the BCC checkbox is not set).
Now you will see the values from the first account in the textbox, instead of the placeholder ("Separate addresses with commas").
So I try to remedy that by clearing the textbox on initial display of the preferences pane (Copies & Folders) of each account: if the value of the textfield was empty initially (.getAttribute("value")) then clear the .value . I don't know if that is correct, but it worked for me satisfactorily. Please test thoroughly.
Attachment #584094 - Attachment is obsolete: true
Attachment #588780 - Flags: ui-review?(bwinton)
Attachment #588780 - Flags: review?(mbanner)
Comment on attachment 588780 [details] [diff] [review]
patch v2

Yeah, that feels better.  Not perfect, but definitely good enough to get a ui-r=me!  :)

Thanks,
Blake.
Attachment #588780 - Flags: ui-review?(bwinton) → ui-review+
Comment on attachment 588780 [details] [diff] [review]
patch v2

Review of attachment 588780 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with the comments fixed.

::: mailnews/base/prefs/content/am-copies.js
@@ +365,5 @@
> +
> +    if (ccChecked) {
> +        if (ccTextbox.value == "") {
> +            ccTextbox.value = document.getElementById("identity.email").value;
> +            if (!init) ccTextbox.select();

nit: put the ccTextbox.select(); on the next line please (same with the bcc function).

@@ +369,5 @@
> +            if (!init) ccTextbox.select();
> +        }
> +    } else {
> +        if ((ccTextbox.value == document.getElementById("identity.email").value)
> +            || (init && ccTextbox.getAttribute("value") == ""))

nit: In mailnews we put the || at the end of the first line. You can also make this into a straight else if.
Attachment #588780 - Flags: review?(mbanner) → review+
(Assignee)

Comment 12

6 years ago
Created attachment 598960 [details] [diff] [review]
patch v3
Attachment #588780 - Attachment is obsolete: true
Attachment #598960 - Flags: review+
(Assignee)

Comment 13

6 years ago
Thanks, fixed.
Keywords: checkin-needed
Whiteboard: [good first bug]
Checked in: http://hg.mozilla.org/comm-central/rev/4fd77b4c57ca
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0
(Assignee)

Updated

6 years ago
Status: RESOLVED → VERIFIED
(Assignee)

Updated

5 years ago
Blocks: 810763
(Assignee)

Comment 15

4 years ago
Test in bug 810763.
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.