155282 - Remove reference to 'drag addresses' for Mailing List dialog.

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Address Book & Contacts, defect, P5)

Description

[[e-mail] (gone - use [e-mail] instead)]

Build ID: Branch 1.0 build (probably on the trunk).

Summary: Remove reference to 'drag addresses' for Mailing List dialog.

Steps to Reproduce:

1.  Have a Mailing List open, and try to drag and drop another list onto the
List's dialog.
2.  Notice that the dialog says 'Type names or drag addresses into the mailing
list below:'

Actual Results:

The problem that you can't drag and drop isn't the issue - the issue here is
that you think you can, because of the text.

Expected Results:

Until we fix D&D, we should remove the reference to it.

Comment 1

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Patch, remove reference to 'drag addresses', and add a comment to reimplement later.

Comment 2

[[e-mail] (gone - use [e-mail] instead)]

Maybe this could only land on the trunk for now...

Comment 3

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Same patch, but with the bug number included in the comments

Comment 4

[(not reading, please use [e-mail] instead)]

Comment on attachment 89864 [details] [diff] [review]
Same patch, but with the bug number included in the comments

sr=sspitzer

thanks for the cleanup.

Comment 5

[racham]

Comment on attachment 89864 [details] [diff] [review]
Same patch, but with the bug number included in the comments

r=bhuvan. Thanks. Get it onto the trunk.

We are way past l10n freeze to get text changes onto the branch. Anyway, adding
Ray to the cc list.

Comment 6

[rchen]

UI change approved from L10N. 

Comment 7

[[e-mail] (gone - use [e-mail] instead)]

This bug was actually the same as bug 138092, the difference is I went ahead and
added the comment into the properties file on the trunk, and I'll try and land
the full changes on the branch.  When I made this fix, I was using the branch tree.

Comment 8

[racham]

BTW, I have noticed that the suggsted text in bug 138092 was

Suggested replacement text: "Type email addresses to add them to the mailing list:"

and the text you have in your patch, for this bug, is 

"Type names into the mailing list below:"

Is the new text (the one in the patch for this bug), approved by Robin/Jennifer
offline..?

Comment 9

[[e-mail] (gone - use [e-mail] instead)]

Ah, good point.  I'd have to say that 'email addresses' is much clearer.

Comment 10

[[e-mail] (gone - use [e-mail] instead)]

I'm pretty sure ADT won't take this - see
http://bugzilla.mozilla.org/show_bug.cgi?id=138092#c10.  Maybe for Buffy, though.

I'm going to go ahead and mark this fixed, since it is on the trunk (same as bug
138092).  If we need this later for Buffy, I can re-evaluate it.  The trunk says
the right thing.

Comment 11

[robinf]

FYI, I'll make sure it's documented in the release notes for now.

Comment 12

[Ninoschka Baca]

Trunk build 2002-07-16: WinMe, Linux RH 7.1, Mac 10.1.3
Verified Fixed.