Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

No welcome page shown for offline installation #10523

Closed
srirambv opened this issue Aug 16, 2017 · 8 comments
Closed

No welcome page shown for offline installation #10523

srirambv opened this issue Aug 16, 2017 · 8 comments
Labels
bug feature/about-pages feature/welcome polish Nice to have — usually related to front-end/visual tasks. priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative).

Comments

@srirambv
Copy link
Collaborator

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    No welcome page shown for offline installation

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    All

  • Brave Version (revision SHA):

Brave 0.18.23
rev 36ae2ec
Muon 4.3.10
  • Steps to reproduce:

    1. Download any 0.18.x packaged build
    2. Go offline and install the setup
    3. Browser launches, about:welcome is a blank page.
  • Actual result:
    See screenshot

  • Expected result:
    Should show about:welcome page even if the machine is offline

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    Yes

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    image

  • Any related issues:
    make welcome screen show up on first time run #9423 Integrate the new welcome screen #7821
    cc: @bradleyrichter @cezaraugusto

@srirambv srirambv added bug feature/about-pages polish Nice to have — usually related to front-end/visual tasks. labels Aug 16, 2017
@srirambv srirambv added this to the 0.21.x (Nightly Channel) milestone Aug 16, 2017
@luixxiul
Copy link
Contributor

luixxiul commented Aug 16, 2017

I noticed it as well, although it should not be the high priority issue as you should be online when you downloaded the package and installed it to run the browser. Unless you move the installer via USB stick to the offline computer or disable the network connection for some reason (which is exceptional because you would not go offline after you downloaded the browser installer; that does not make sense in the real world), this won't be a great issue.

Still I think it would be a good idea to include the about page as a offline page for faster start up with a slow network connection, avoiding rendering from being blocked.

@luixxiul luixxiul removed this from the 0.21.x (Nightly Channel) milestone Aug 16, 2017
@srirambv
Copy link
Collaborator Author

disable the network connection for some reason (which is exceptional because you would not go offline after you downloaded the browser installer; that does not make sense in the real world)

I don't think this is exceptional at all. User can download and decide to install at a later point of time and may be offline at that time or might have network issues, in which case this becomes a real problem and is not a good first experience. It should not be expected that the user has to be online just because he/she downloaded the setup. I think this should be included in packaged build soon.

cc: @bradleyrichter for thoughts

@luixxiul
Copy link
Contributor

decide to install at a later point of time and may be offline at that time

I simply could not imagine which situations are common where you would like to install a Internet browser, being conscious that the network connection is offline.. There could be a person who would install the browser on a train / in a cafe where the Internet connection is not available, and that can be the case here.

Opening the browser for the first time, you would expect something but a white tab would be displayed. The welcome page obviously should be displayed on any case, and I'm not against for it at all.

@bradleyrichter
Copy link
Contributor

bradleyrichter commented Aug 16, 2017 via email

@evq
Copy link
Member

evq commented Oct 13, 2017

bump! noticed into this when updating in a qubes templatevm (restricted internet access)

@cezaraugusto
Copy link
Contributor

is the welcome page hosted as a WP page or just a redirect from another source? we could cache contents w/ a service worker for offline support or maybe just put some message as a fallback in the browser when the page is unavailable.

@bsclifton bsclifton added the priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). label Oct 16, 2017
@bsclifton
Copy link
Member

bsclifton commented Oct 16, 2017

@cezaraugusto I think a good solution would be to (during the build process) do a wget for the page and store the HTML in a static location, which would then be accessible once Brave is installed. We could always fall back to that version if the internet is not available

We should have detection code for network up / network down (originally implemented for use w/ Ledger)

As @bradleyrichter mentioned, I also think this is a lower priority issue. I tagged it with P5

@bsclifton bsclifton added this to the Backlog (Prioritized) milestone Nov 22, 2017
@srirambv
Copy link
Collaborator Author

srirambv commented Jun 5, 2018

Closing the issue as #12691 got closed as fixed in brave-core brave/brave-core#130 (comment)

@srirambv srirambv closed this as completed Jun 5, 2018
@srirambv srirambv removed this from the Backlog (Prioritized) milestone Jun 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug feature/about-pages feature/welcome polish Nice to have — usually related to front-end/visual tasks. priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative).
Projects
None yet
Development

No branches or pull requests

7 participants