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

"Brave starts with Dashboard" incompatible with "A new tab shows Blank page" #6521

Closed
Zuzu-Typ opened this issue Jan 4, 2017 · 7 comments
Closed

Comments

@Zuzu-Typ
Copy link

Zuzu-Typ commented Jan 4, 2017

Did you search for similar issues before submitting this one?
I did a quick search

Describe the issue you encountered:
Brave doesn't always start the Dashboard, even though it is selected in the preferences.

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

  • Brave Version (revision SHA):
    Brave 1.13.0dev RC5

  • Steps to reproduce:

    1. Go to about:preferences
    2. select Dashboard for Brave starts with
    3. select something other than Dashboard for A new tab shows
    4. restart Brave
  • Screenshot if needed:

  • Any related issues:

@Zuzu-Typ Zuzu-Typ changed the title Starting Brave with the dashboard doesn't always open the dashboard Starting Brave with the dashboard opens a new tab instead Jan 22, 2017
@Zuzu-Typ
Copy link
Author

Zuzu-Typ commented Jan 22, 2017

Here's an example scenario:
Here are the settings:
blankpage

On restarting, this is what Brave should do (open the Dashboard):
braveshoulddo

This is what Brave does though (open a new tab):
bravedoes

@luixxiul
Copy link
Contributor

luixxiul commented Jan 22, 2017

Nice find!

@bbondy does this block the release?

@bbondy bbondy added this to the 0.13.0 milestone Jan 22, 2017
@bsclifton bsclifton self-assigned this Jan 23, 2017
@bsclifton
Copy link
Member

bsclifton commented Jan 23, 2017

Unable to reproduce with master; I believe this was fixed with d4f79be (RC5 also had an older version of Muon).

Let's re-open if this occurs in the new RC

edit the step I missed was picking "A new tab shows: Blank page". I went off original post which said pick anything except "Dashboard"

@bsclifton bsclifton removed this from the 0.13.0 milestone Jan 23, 2017
@bsclifton bsclifton reopened this Jan 23, 2017
@bsclifton bsclifton added this to the 0.13.0 milestone Jan 23, 2017
@bsclifton
Copy link
Member

Re-opening after @NejcZdovc was able to reproduce

@bsclifton bsclifton changed the title Starting Brave with the dashboard opens a new tab instead "Brave starts with Dashboard" incompatible with "A new tab shows Blank page" Jan 23, 2017
@bsclifton bsclifton modified the milestones: 0.13.1, 0.13.0 Jan 23, 2017
@bsclifton
Copy link
Member

bsclifton commented Jan 23, 2017

After implementing a "fix" for this, I strongly believe the fix would introduce unnecessary complexity.

I think we have two better choices we can make:

  1. Don't allow folks to pick "Brave starts with Dashboard" if "A new tab shows" is set to "Blank page"
  2. Break dashboard out into it's own page, about:dashboard. There would no longer be a conflict, but about:newtab and about:dashboard would be two separate pages.

Both options get rid of the conflict. Letting someone launch into dashboard where they can see stats and tiles is useless IMO if there's not a way for them to get back (because new tabs would only show blank).

My personal preference would be choice 1 for now. We could create an issue for choice 2 and push to the contributor backlog

cc: @bradleyrichter @cezaraugusto @NejcZdovc

@bsclifton
Copy link
Member

bsclifton commented Jan 23, 2017

Talked with @bradleyrichter and we decided to chose another option:

I marked as release notes exclude since this new "dashboard" choice was introduced with 0.13.0 with #5690

@bsclifton
Copy link
Member

Test Plan

  1. Go to about:preferences
  2. Ensure "Brave starts with" dropdown no longer has "Dashboard"; it should have "a new tab"

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants