Open
Bug 800321
Opened 12 years ago
Updated 2 years ago
checking for updates in the About dialog does not handle Offline mode
Categories
(Thunderbird :: General, defect)
Thunderbird
General
Tracking
(Not tracked)
NEW
People
(Reporter: aryx, Assigned: aryx)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mkmelin
:
review-
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #701733 +++
When Firefox (or Thunderbird) is in offline mode, going into the Help->About dialog still shows the "check for updates" button (or checks automatically). The network is not hit therefore no updates are found and the message says FF (or TB) is up to date (which may not be true).
I was hit by this in some test profiles where I wanted to be in permanent offline mode (but already forgot I have set it in the past), but still wanted nightly updates. The message misled me into thinking there really weren't any updates.
Expected results:
Choose any solution out of:
1. no button or automatic check. Instead a message the program is in offline mode in its place.
2. or allow to run check but return "the program is in offline mode" instead "program is up to date".
Assignee | ||
Comment 1•12 years ago
|
||
Suggested patch to check if Thunderbird is offline, and if yes, offer a button to go online and check for updates.
Please decide if the "Update" in the string should be lowercase.
Try run: https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=6816a0a25434
Attachment #670388 -
Flags: review?(mbanner)
Comment 2•12 years ago
|
||
Comment on attachment 670388 [details] [diff] [review]
check for offline in aboutDialog
Sorry for the delay in this, Magnus or Mike would probably be able to handle this faster at the moment.
Attachment #670388 -
Flags: review?(mbanner) → review?(mkmelin+mozilla)
Comment 3•12 years ago
|
||
Comment on attachment 670388 [details] [diff] [review]
check for offline in aboutDialog
Review of attachment 670388 [details] [diff] [review]:
-----------------------------------------------------------------
I think we should just check if we're online when the user clicks the button, and ask to go online then. Or are there a good reason not to do it like that?
::: mail/locales/en-US/chrome/messenger/aboutDialog.dtd
@@ +29,5 @@
> <!-- LOCALIZATION NOTE (update.noUpdatesFound): try to make the localized text short (see bug 596813 for screenshots). -->
> <!ENTITY update.noUpdatesFound "&brandShortName; is up to date">
> +<!-- LOCALIZATION NOTE (update.offline): try to make the localized text short (see bug 596813 for screenshots). -->
> +<!ENTITY update.offline "&brandShortName; needs to be online to check for Updates.">
> +<!ENTITY update.goOnline "Go online.">
Go Online
Attachment #670388 -
Flags: review?(mkmelin+mozilla) → review-
Assignee | ||
Comment 4•12 years ago
|
||
(In reply to Magnus Melin from comment #3)
> I think we should just check if we're online when the user clicks the
> button, and ask to go online then. Or are there a good reason not to do it
> like that?
So do you want a prompt like the one which is shown when Thunderbird launches and should ask if it should go online or offline? The about dialog itself automatically checks for updates when opened, requiring closing and reopening it after switching to online at the moment.
> ::: mail/locales/en-US/chrome/messenger/aboutDialog.dtd
> > +<!ENTITY update.goOnline "Go online.">
>
> Go Online
This is not in line with the other strings there, only subjects start with uppercase: http://mxr.mozilla.org/comm-central/source/mail/locales/en-US/chrome/messenger/aboutDialog.dtd
Comment 5•12 years ago
|
||
(In reply to Archaeopteryx [:aryx] from comment #4)
> (In reply to Magnus Melin from comment #3)
> > I think we should just check if we're online when the user clicks the
> > button, and ask to go online then. Or are there a good reason not to do it
> > like that?
> So do you want a prompt like the one which is shown when Thunderbird
> launches and should ask if it should go online or offline? The about dialog
> itself automatically checks for updates when opened, requiring closing and
> reopening it after switching to online at the moment.
Something similar yes.
If you're offline and open it it shouldn't of course tell you you are "up to date". Maybe we should just show the Check for Updates button in that case too.
> > ::: mail/locales/en-US/chrome/messenger/aboutDialog.dtd
> > > +<!ENTITY update.goOnline "Go online.">
> >
> > Go Online
> This is not in line with the other strings there, only subjects start with
> uppercase:
> http://mxr.mozilla.org/comm-central/source/mail/locales/en-US/chrome/
> messenger/aboutDialog.dtd
Those other examples aren't button labels. Compare to "Check for Updates"
Updated•3 years ago
|
Comment 7•3 years ago
|
||
Looks like incarnations of this bug are scattered all over the place.
Maybe the one with the highest chances of success is currently my bug 1643309, which has "recently" (Feb-2021) reviewed patches, and might just be missing a test. Guess we'll cleanup after the deed is done.
Status: ASSIGNED → NEW
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•