"Firefox is in offline mode" is misleading when there is no network connection

RESOLVED FIXED in Firefox 24

Status

()

Firefox for Android
General
RESOLVED FIXED
6 years ago
a year ago

People

(Reporter: gerv, Assigned: Bruce Wu)

Tracking

({regression, uiwanted, ux-jargon})

14 Branch
Firefox 24
regression, uiwanted, ux-jargon
Points:
---

Firefox Tracking Flags

(blocking-fennec1.0 -, fennec+)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
If I don't have a network connection and try and visit a site, it tells me "Firefox is in offline mode". While perhaps technically true, this is spectacularly unhelpful, because it leads me to look for some sort of switch within Firefox to switch to "online" mode, like there is on Desktop, while wondering how on earth it got into that state.

A better message would be something like "Firefox cannot access the Internet. Please check that you have a working Internet connection. [ Try Again ]".

Gerv
tracking-fennec: --- → ?
tracking-fennec: ? → +
Keywords: uiwanted
blocking-fennec1.0: --- → ?
Not blocking release on this, but we'll continue tracking this bug.
blocking-fennec1.0: ? → -

Comment 2

5 years ago
I wonder why we're getting this error page instead of the "Server not found" error page that shows up on desktop.

If we haven't done this intentionally, it seems like a bug somewhere it our networking code, rather than in the error page itself.
In all the months passed, I havn't seen this; although there is a recent report here which I will dupe in a second.

Updated

5 years ago
Duplicate of this bug: 845356

Updated

5 years ago
Keywords: steps-wanted

Comment 5

5 years ago
I reproduced this by turning off my wifi and trying to navigate to a webpage.

Comment 6

5 years ago
I'm not 100% certain that my bug 845356 IS a duplicate of this.

I actually HAVE a network connection (can ping my device and can visit other websites using the default browser).

But when I take the option to sync from the menu, I get a message "No internet connection available".

And (apparently randomly) trying to visit websites gives "Firefox is currently in offline mode and can't browse the web". Though weirdly, at the present moment, although I'm unable to sync, I can actually get websites. But ten minutes ago I couldn't.

My device is currently sitting no more than 20cm from my WiFi router and is (not surprisingly) showing high signal strength.

Comment 7

5 years ago
Update. I can stop this behaviour by following the instructions at http://superuser.com/questions/208146/can-firefoxs-offline-mode-be-disabled

Just in case it helps anybody else who lands here looking for a solution.
(Assignee)

Comment 8

4 years ago
I am wondering if anyone is working on this bug, if not I would love to contribute some development time into it. 

If anyone have editing privileges see this, please assign the bug to me. 

Thanks~
(In reply to Bruce Wu from comment #8)
> If anyone have editing privileges see this, please assign the bug to me. 

Done; thanks for spending time on this! If you have any questions about the bug, you can ask them here, or you can try asking in #ux or #fx-team on irc.mozilla.org.
Assignee: nobody → brucewu

Comment 10

4 years ago
From what I can see, it seems like Firefox for Android doesn't have offline mode; it just states that it's in "offline mode" when it has no network connection. If so, why not just change the error message in chrome/overrides/appstrings.properties?

This has been a problem before as well, and it seems like they just overrided the string: bug 523296.
(Assignee)

Comment 11

4 years ago
(In reply to David Hsu from comment #10)
> This has been a problem before as well, and it seems like they just
> overrided the string: bug 523296.

So far I am trying to reproduce Tony Green's finding (comment #6). Otherwise,
I agree with you that this is simply the matter of word choice.
(Assignee)

Comment 12

4 years ago
Created attachment 742783 [details] [diff] [review]
change description of netError to match connectionFailure

I changed the message content instead of inspect all statement involve netError to see if they should be substitute with connectionFailure. 

Since Fennec do not have an "Offline mode" right now, changing the message is ok.. but in the future when we do introduce Offline Mode .. this will be a problem.
Attachment #742783 - Flags: review?(mbrubeck)
Blocks: 729951
Keywords: regression
Comment on attachment 742783 [details] [diff] [review]
change description of netError to match connectionFailure

Review of attachment 742783 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for the patch!  The new strings look good; but unfortunately this version of the patch will only affect English-language builds.  Since it keeps the same names for the strings, other locales will continue using their current strings.

Normally we would just change each string name to something like "netOffline2.title" and then making the corresponding change in the file that uses it:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/netError.xhtml

...but in this case, one of the strings is used in Gecko code that's shared with other platforms (B2G, desktop) so we can't easily change it just for mobile.  This is a known problem with the way we do string overrides:
http://mxr.mozilla.org/mozilla-central/source/docshell/base/nsDocShell.cpp#4288

I'm not sure yet of the best way to fix this...  We might consider changing the string for all platforms, or adding a separate string for cases (like Android) where "network.manage-offline-status" is enabled.
Attachment #742783 - Flags: review?(mbrubeck) → review-
Keywords: steps-wanted → ux-jargon
Version: unspecified → Firefox 14
(Assignee)

Comment 14

4 years ago
Created attachment 743250 [details] [diff] [review]
patch_v2 - this patch affect all locale

Since the shortDesc cannot be change easily without effecting other platform, I left it alone and changed only the title / longDesc. This way at least the user will know the situation is "Unable to Connect", and the description also suggest user to check for data/wifi connection.
Attachment #742783 - Attachment is obsolete: true
Comment on attachment 743250 [details] [diff] [review]
patch_v2 - this patch affect all locale

Review of attachment 743250 [details] [diff] [review]:
-----------------------------------------------------------------

I like this; it seems like a good pragmatic compromise.  If you think this is ready to be reviewed and checked in, please set the review? flag and enter :mfinkle as the reviwer.
Attachment #743250 - Flags: feedback+
(Assignee)

Updated

4 years ago
Attachment #743250 - Flags: review?(mark.finkle)
Comment on attachment 743250 [details] [diff] [review]
patch_v2 - this patch affect all locale

Agreed. This seems like better text to display to the user.
Attachment #743250 - Flags: review?(mark.finkle) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/3a2756741ff0
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/3a2756741ff0
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 24
You need to log in before you can comment on or make changes to this bug.