Closed
Bug 335266
Opened 19 years ago
Closed 16 years ago
If update of Thunderbird was paused, incorrect message is shown
Categories
(Toolkit :: Application Update, defect)
Toolkit
Application Update
Tracking
()
RESOLVED
FIXED
mozilla1.9.1b2
People
(Reporter: unghost, Assigned: unghost)
Details
Attachments
(2 files, 3 obsolete files)
2.37 KB,
patch
|
robert.strong.bugs
:
review+
|
Details | Diff | Splinter Review |
2.36 KB,
patch
|
Details | Diff | Splinter Review |
If update of Thunderbird was paused, user sees message: "You have paused downloading this update. Do you want %S to download the update in the background while you continue to browse?"
The problem that you can not browse with Thunderbird - it's mail application. Obviously this phrase was written with Firefox in mind, though this part of localization belongs to Toolkit.
Patch is following.
Assignee | ||
Updated•19 years ago
|
Attachment #219614 -
Flags: review? → review?(mconnor)
Comment 2•19 years ago
|
||
"while you continue to work with application" doesn't seem quite right. Maybe just "while you continue to work"?
Assignee | ||
Comment 3•19 years ago
|
||
(In reply to comment #2)
> "while you continue to work with application" doesn't seem quite right. Maybe
> just "while you continue to work"?
>
I'm not native English speaker, so you should know better. Do you want me to make new patch or I should wait for more input from mconnor/beltzner?
Assignee | ||
Comment 4•19 years ago
|
||
With comments from Gavin addressed.
Attachment #219614 -
Attachment is obsolete: true
Attachment #220030 -
Flags: review?(mconnor)
Attachment #219614 -
Flags: review?(mconnor)
Comment 5•18 years ago
|
||
The latest update (yesterday) cannot be installed and consequently prevents me from accessing my thunderbird email. very serious for me
Updated•16 years ago
|
Product: Firefox → Toolkit
Comment 6•16 years ago
|
||
Comment on attachment 220030 [details] [diff] [review]
Proposed patch v.2
this needs to change the string name, here and in the consumers, so that localizers re-translate it as well.
Attachment #220030 -
Flags: review?(mconnor) → review-
Comment 7•16 years ago
|
||
Hey Boriss, could you give input on this string change? I would prefer something that didn't mention work.
-resumePausedAfterCloseMessage=You have paused downloading this update. Do you want %S to download the update in the background while you continue to browse?
+resumePausedAfterCloseMessage=You have paused downloading this update. Do you want %S to download the update in the background while you continue to work?
Comment 8•16 years ago
|
||
Alexander, please hold off on this change until a new string is provided by Boriss. At that time a new patch can be created with that string and the resumePausedAfterCloseMessage property name will need to be changed so localizers will see that the string has changed. Thanks
Comment 9•16 years ago
|
||
My suggestion is attached. I agree with Rob, that implying the user is doing a particular task (like work) does not make sense with an application like Thunderbird. The verb "browse" for a browser essentially fits any Firefox-related task, but for an application the only action we know the user is taking is using that application. So, I changed "work" to "use."
Attachment #220030 -
Attachment is obsolete: true
Assignee | ||
Comment 10•16 years ago
|
||
Patch with Jennifer Boriss' text. Property name is changed.
Attachment #343639 -
Flags: review?(robert.bugzilla)
Updated•16 years ago
|
Attachment #343639 -
Flags: review?(robert.bugzilla) → review+
Comment 11•16 years ago
|
||
Comment on attachment 343639 [details] [diff] [review]
Patch v.3
Looks good... just use resumePausedAfterCloseMsg for the name. Thanks!
Assignee | ||
Comment 12•16 years ago
|
||
Assignee: nobody → unghost
Status: NEW → ASSIGNED
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Updated•16 years ago
|
Attachment #343626 -
Attachment is obsolete: true
Comment 13•16 years ago
|
||
Comment on attachment 344133 [details] [diff] [review]
Patch for checkin
[Checkin: Comment 13]
http://hg.mozilla.org/mozilla-central/rev/f5593ad6dcc0
Attachment #344133 -
Attachment description: Patch for checkin → Patch for checkin
[Checkin: Comment 13]
Updated•16 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: in-testsuite-
Keywords: checkin-needed
OS: Windows XP → All
Hardware: PC → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1b2
Version: unspecified → Trunk
You need to log in
before you can comment on or make changes to this bug.
Description
•