Closed Bug 335266 Opened 18 years ago Closed 16 years ago

If update of Thunderbird was paused, incorrect message is shown

Categories

(Toolkit :: Application Update, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.1b2

People

(Reporter: unghost, Assigned: unghost)

Details

Attachments

(2 files, 3 obsolete files)

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.
Attached patch Proposed patch (obsolete) — Splinter Review
Proposed patch.
Attachment #219614 - Flags: review?
Attachment #219614 - Flags: review? → review?(mconnor)
"while you continue to work with application" doesn't seem quite right. Maybe just "while you continue to work"?
(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? 
Attached patch Proposed patch v.2 (obsolete) — Splinter Review
With comments from Gavin addressed.
Attachment #219614 - Attachment is obsolete: true
Attachment #220030 - Flags: review?(mconnor)
Attachment #219614 - Flags: review?(mconnor)
The latest update (yesterday) cannot be installed and consequently prevents me from accessing my thunderbird email.  very serious for me
Product: Firefox → Toolkit
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-
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?
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
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
Attached patch Patch v.3Splinter Review
Patch with Jennifer Boriss' text. Property name is changed.
Attachment #343639 - Flags: review?(robert.bugzilla)
Attachment #343639 - Flags: review?(robert.bugzilla) → review+
Comment on attachment 343639 [details] [diff] [review]
Patch v.3

Looks good... just use resumePausedAfterCloseMsg for the name. Thanks!
Assignee: nobody → unghost
Status: NEW → ASSIGNED
Keywords: checkin-needed
Attachment #343626 - Attachment is obsolete: true
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]
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.

Attachment

General

Created:
Updated:
Size: