208628 - Copies & Folders: Initialize Bcc field to the current address

Status: VERIFIED FIXED

(MailNews Core :: Account Manager, enhancement, P2)

Description

[Bruce K Gillespie]

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

[Mike Cowperthwaite]

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

Comment 2

[Mike Cowperthwaite]

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.

Comment 3

[Gervase Markham [:gerv]]

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/

Comment 4

[Gervase Markham [:gerv]]

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.

Comment 5

[Karsten Düsterloh]

Still valid and reasonable.

Comment 6

[:aceman]

It seems I could try to patch this.

Comment 7

[:aceman]

Attachment: init BCC and CC if selected, clear them if deselected

Comment 8

[Blake Winton (:bwinton) (:☕️)]

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.

Comment 9

[:aceman]

Attachment: 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.

Comment 10

[Blake Winton (:bwinton) (:☕️)]

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.

Comment 11

[Mark Banner (:standard8)]

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.

Comment 12

[:aceman]

Attachment: patch v3

Comment 13

[:aceman]

Thanks, fixed.

Comment 14

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/4fd77b4c57ca

Comment 15

[:aceman]

Test in bug 810763.