Closed Bug 1280599 Opened 8 years ago Closed 4 years ago
Handle captive portal page not loading
Captive Portal Service detects we are in a captive portal. UI opens a new tab to the captive portal endpoint which should get redirected. If the network is flaky/bad we might end up with an empty tab. While it's loading the user has no idea what it's for. The captive portal tab should be hidden until it's fully loaded. (if possible)
I don't think we should hide it. I think we should make it obvious there is a captive portal as soon as we know there is one. A better idea in my opinion is to make it very clear to the user what the tab is for, which the notification bar will do (or doorhanger if we decide to switch to that in the future). MattN, what do you think?
Right, once we have the doorhanger/notification bar this issue isn't exactly relevant anymore. However, we still need to handle the tab not loading (due to a timeout or some other issue), in some way.
Another idea would be to show something meaningful instead of a blank page while loading.
I also think we should handle this with the notification bar. If this bug is about handling the page not loading then the summary should be changed.
Summary: Hide captive portal tab while it's loading → Handle captive portal page not loading
No empty tab, no indication other than bad cert warning on every open tab attempting to connect to WiFi on Scot Rail and Virgin Trains in Scotland and England this week. Using Firefox 49.0a2 (Dev. Ed.) and FF for iOS 4.0(11) Also no way to open a tab and get to the portal authentication page. Had to use Safari or Chrome on OS X and Safari on iOS.
The captive-portal UI work is just starting to land on nightly. It's not in aurora yet.
Component: Networking → General
Product: Core → Toolkit
Version: unspecified → Trunk
If the captive portal is such a crappy network that it can't even load the login page, isn't it enough to treat the tab the same as a regular page load (i.e. spinner keeps spinning, at some point the user hits reload)?
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.