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
Not blocking release on this, but we'll continue tracking this bug.
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.
I reproduced this by turning off my wifi and trying to navigate to a webpage.
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.
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.
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.
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.
(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.
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.
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.
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.
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.
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.