Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Tab override via extension not working #993

Closed
petemill opened this issue Sep 6, 2018 · 23 comments · Fixed by brave/brave-core#933
Closed

New Tab override via extension not working #993

petemill opened this issue Sep 6, 2018 · 23 comments · Fixed by brave/brave-core#933

Comments

@petemill
Copy link
Member

petemill commented Sep 6, 2018

Description

Extensions are able to override a bult-in page such as New Tab page https://developer.chrome.com/extensions/override
This works in Chrome but not in Brave 0.55

Steps to Reproduce

  1. Install momentum extension from https://chrome.google.com/webstore/detail/momentum/laookkfknpbbblfpciffpaejjkokdgca/related?hl=en
  2. Open New Tab

Actual result:

Brave's default New Tab page is shown

Expected result:

Momentum's New Tab page is shown

Reproduces how often:

Always

Brave version (about:brave info)

0.55.1 Chromium: 70.0.3528.4 (Official Build) dev (64-bit)

Additional

Brave policy is to allow these overrides, so this is a bug

@petemill petemill added this to the 1.x Backlog milestone Sep 6, 2018
@eljuno
Copy link

eljuno commented Sep 20, 2018

+1 from community (likely with other extension) https://community.brave.com/t/home-page-new-tab-page/32776?u=eljuno

@ngurney
Copy link

ngurney commented Sep 26, 2018

+1 for me as well. Looking to use the Tabliss extention which I assume is caused by the same issue.

@srirambv
Copy link
Contributor

srirambv commented Oct 9, 2018

+1 from https://community.brave.com/t/custom-new-tab-page-settings/34468

@Rejinderi
Copy link

+1, need this for Refind new tab before i can totally put chrome aside.

@srirambv
Copy link
Contributor

+1 from @krono via #1771

@andrewcking
Copy link

I think #960 will accomplish this ("New Tab Page Shows")

@eljuno
Copy link

eljuno commented Oct 25, 2018

@srirambv
Copy link
Contributor

+1 from @krmbzds via #1872

@KyleConstantin
Copy link

+1 this is a major issue for me converting fully over to brave. please fix!

@krmbzds
Copy link

krmbzds commented Oct 28, 2018

Why not move new tab feature to a Brave New Tab extension?

Advantages:

  • Better Chromium parity
  • More plugin compatibility
  • Less maintenance (?)

And anyone who doesn't want it would be free to disable or substitute it.

@bsclifton
Copy link
Member

cc: @rebron for prioritization

@bbondy bbondy added the priority/P2 A bad problem. We might uplift this to the next planned release. label Nov 5, 2018
@bbondy
Copy link
Member

bbondy commented Nov 5, 2018

I put it to P2 because there's a lot of community asks for it. We can do this soon.

@rebron rebron added this to P1 & P2 Backlog in General Nov 5, 2018
@Brave-Matt
Copy link
Collaborator

FWIW: This is easily one of the most frequently requested/mentioned features users ask for. Both the ability to override NTP with extensions and for Brave NTP redesign.
While I clearly understand the potential workload involved in redesigning our own new tab page/dashboard, I think its priority merits reconsideration. Users should obviously be able to use any of the available extensions if they want but I don't think installing 3p extensions is ideal.

@bbondy
Copy link
Member

bbondy commented Nov 19, 2018

@Brave-Matt this is a different issue than new tab redesign and this is already marked with top priority. It'll be done soon. Reconsidering priority wouldn't help since it's already marked with top priority. P1 is reserved for something we need to do a chemspill release for.

bbondy added a commit to brave/brave-core that referenced this issue Nov 19, 2018
bbondy added a commit to brave/brave-core that referenced this issue Nov 19, 2018
bbondy added a commit to brave/brave-core that referenced this issue Nov 20, 2018
bbondy added a commit to brave/brave-core that referenced this issue Nov 20, 2018
@bbondy bbondy added the QA/Yes label Nov 20, 2018
@bbondy bbondy self-assigned this Nov 20, 2018
@bbondy
Copy link
Member

bbondy commented Nov 20, 2018

Fixed here, in-review now:
brave/brave-core#933

bbondy added a commit to brave/brave-core that referenced this issue Nov 20, 2018
General automation moved this from P1 & P2 Backlog to Completed Nov 20, 2018
@bbondy bbondy modified the milestones: 1.x Backlog, 0.58.x - Dev Nov 21, 2018
@bbondy bbondy added this to Completed in Shields Nov 21, 2018
@bbondy bbondy removed this from Completed in Shields Nov 21, 2018
@manitaggarwal
Copy link

Still not working for me..

@bbondy
Copy link
Member

bbondy commented Dec 5, 2018

@manitaggarwal are you using 0.57.x? It'll be released tomorrow but you can get an early RC on github already.

@bbondy
Copy link
Member

bbondy commented Dec 5, 2018

Oops sorry this is only on 0.58.x so better to grab a Dev channel build from the releases page on this repo.

@manitaggarwal
Copy link

@bbondy I am on 0.56, I'll wait for this fix to be merged into stable channel... Thanks for the info.. 😄

@xverz
Copy link

xverz commented Dec 9, 2018

Can't wait for this to be implemented to my favorite browser.

@srirambv
Copy link
Contributor

srirambv commented Dec 11, 2018

Verification passed on

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta(64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Linux
  • screenshot from 2018-12-11 15-16-24

Verification passed on

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Windows
  • Verified the STR from description

Verified passed with

Brave 0.58.12 Chromium: 71.0.3578.80 (Official Build) (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Mac OS X
  • Verified STR from description
    screen shot 2018-12-13 at 12 00 39 pm

@Zelrain
Copy link

Zelrain commented Dec 15, 2018

Need to be able to customize new tab page! You already know!

@dogancelik
Copy link

Brave's default New Tab extension overrides my Chrome theme's wallpaper!

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

Successfully merging a pull request may close this issue.