Closed
Bug 614407
Opened 14 years ago
Closed 13 years ago
Explanation of "reply to" field would be helpful
Categories
(Thunderbird :: Account Manager, defect)
Thunderbird
Account Manager
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 9.0
People
(Reporter: mozilla, Assigned: jyeo)
Details
(Whiteboard: [good first bug])
Attachments
(1 file, 1 obsolete file)
4.80 KB,
patch
|
gkw
:
review+
gkw
:
ui-review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_8; en-US) AppleWebKit/534.7 (KHTML, like Gecko) Chrome/7.0.517.44 Safari/534.7
Build Identifier: 3.1.6
The manage identities>Edit Identity> "Reply to address" section should have a note saying "Leave this blank unless you explicitly want email replies to go to a different address".
Reproducible: Always
I haven't been able to figure out why my name hasn't been appearing in replies I get and deleting the "reply to" address altogether fixed the problem!
Comment 1•14 years ago
|
||
Bryan thoughts ?
Comment 2•14 years ago
|
||
Seems like a good idea, I assume lots of people fill this in for the wrong reasons.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•14 years ago
|
Whiteboard: [good first bug]
Assignee | ||
Comment 3•13 years ago
|
||
Attachment #556744 -
Flags: ui-review?(bwinton)
Attachment #556744 -
Flags: review?(dbienvenu)
Assignee | ||
Updated•13 years ago
|
Attachment #556744 -
Attachment description: edited the following files: → edited the following files:
+++ b/mail/locales/en-US/chrome/messenger/am-main.dtd
+++ b/mailnews/base/prefs/content/am-identity-edit.xul
+++ b/mailnews/base/prefs/content/am-main.xul
+++ b/suite/locales/en-US/chrome/mailnews/pref/am-main.dtd
Assignee | ||
Updated•13 years ago
|
Attachment #556744 -
Attachment description: edited the following files:
+++ b/mail/locales/en-US/chrome/messenger/am-main.dtd
+++ b/mailnews/base/prefs/content/am-identity-edit.xul
+++ b/mailnews/base/prefs/content/am-main.xul
+++ b/suite/locales/en-US/chrome/mailnews/pref/am-main.dtd → 614407#1.patch
Attachment #556744 -
Attachment filename: replyToPatch → 614407#1.patch
Comment 4•13 years ago
|
||
Comment on attachment 556744 [details] [diff] [review]
614407#1.patch
I think it's better, my only complaint is that it sounds a little like we're going to send all replies the user sends to that address, instead of asking other people to send replies to the user to a different address.
The text "Ask other people to reply to this other address" also seems to fit, and is closer to what I think we're trying to say, but I'm happy to hear more suggestions. :)
ui-r=me with the new text.
Thanks,
Blake.
Attachment #556744 -
Flags: ui-review?(bwinton) → ui-review+
Assignee | ||
Comment 5•13 years ago
|
||
> The text "Ask other people to reply to this other address" also seems to
> fit, and is closer to what I think we're trying to say, but I'm happy to
> hear more suggestions. :)
current:
"send email replies to this address"
how about these:
"Recipients will send replies to this address"
"Your recipients will reply to this address"
"Your recipients will send replies to this address"
"Recipients will reply to this address" (i prefer this, it's shorter)
"Recipients will reply to this other address"
"Your recipients will reply to this other address"
Comment 6•13 years ago
|
||
(In reply to Jason(:jyeo) from comment #5)
> "Recipients will reply to this address" (i prefer this, it's shorter)
It is shorter, but I think it still suffers some of the confusion about potentially being the same as their regular address, which is why I would prefer:
> "Recipients will reply to this other address"
(I'm also assigning this to you, cause it looks like you're the one who'll fix it. :)
Thanks,
Blake.
Assignee: nobody → jasonyeo88
Status: NEW → ASSIGNED
Comment 7•13 years ago
|
||
Comment on attachment 556744 [details] [diff] [review]
614407#1.patch
thx for the patch
Attachment #556744 -
Flags: review?(dbienvenu) → review+
Updated•13 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 8•13 years ago
|
||
Attachment #556744 -
Attachment is obsolete: true
Attachment #557734 -
Flags: ui-review?(bwinton)
Attachment #557734 -
Flags: review?(dbienvenu)
Assignee | ||
Comment 9•13 years ago
|
||
Comment on attachment 557734 [details] [diff] [review]
bug614407#2.patch
i've changed the text to Recipients will reply to this other address, as requested by blake. :)
Attachment #557734 -
Attachment description: i've changed the text as requested by blake → bug614407#2.patch
Comment 10•13 years ago
|
||
Comment on attachment 557734 [details] [diff] [review]
bug614407#2.patch
r+ from bienvenu and ui-r+ from bwinton (with the change in wording done in this patch) carried over.
Attachment #557734 -
Flags: ui-review?(bwinton)
Attachment #557734 -
Flags: ui-review+
Attachment #557734 -
Flags: review?(dbienvenu)
Attachment #557734 -
Flags: review+
Comment 11•13 years ago
|
||
Checked in: http://hg.mozilla.org/comm-central/rev/814036585e36
Thanks Jason.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 9.0
You need to log in
before you can comment on or make changes to this bug.
Description
•