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

Cannot remove top sites from home page #3852

Closed
jlrnr opened this issue Mar 22, 2019 · 26 comments
Closed

Cannot remove top sites from home page #3852

jlrnr opened this issue Mar 22, 2019 · 26 comments

Comments

@jlrnr
Copy link

jlrnr commented Mar 22, 2019

Description

I am unable to remove any of the 'top sites' thumbnails from the homepage. I have the option to close them, however, on clicking I receive the usual message popup "Top Site Removed". Thumbnail stay in place, none of the additional options 'undo' or 'restore all' do anything either.

Steps to Reproduce

  1. Attempt to remove any of the top site icons.

Actual result:

All thumbnails remain on the page.

Expected result:

The top sites should be able to be modified.

Reproduces how often:

This has been an issue through several versions

Brave version (brave://version info)

Version 0.61.52 64bit
Windows 10 Home 64bit

@rebron
Copy link
Collaborator

rebron commented Mar 22, 2019

cc: @kjozwiak to check regression range

@rebron
Copy link
Collaborator

rebron commented Mar 29, 2019

cc: @cezaraugusto

@rebron rebron added priority/P2 A bad problem. We might uplift this to the next planned release. priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P2 A bad problem. We might uplift this to the next planned release. labels Mar 29, 2019
@stixjohno
Copy link

I have this very same issue, I cannot remove the Topsites Icon, I have 4 of the same, so not even top site

@cezaraugusto
Copy link
Contributor

not related to new changes for sure as they only landed starting 0.62.x. however we should probably have a new tab initiative to cover some top sites bugs as they exist and are annoying. I'm self-assigning this one

@cezaraugusto cezaraugusto self-assigned this Apr 1, 2019
@cezaraugusto cezaraugusto assigned imptrx and unassigned cezaraugusto May 29, 2019
@cezaraugusto
Copy link
Contributor

assigning @imptrx and un-assigning myself. cc @rebron I think the new mechanism for top sites in NTP re-work will eventually close this as invalid but please coordinate as needed

@mobbdeep
Copy link

I'm having this issue too.

@lee-b
Copy link

lee-b commented Jul 2, 2019

+1. Note that there was a previous bug similar to this:

brave/browser-laptop#10411

@tgomes95
Copy link

tgomes95 commented Jul 25, 2019

I am having the same issue.

Brave Version (brave://version)

Version 0.66.101 Chromium: 75.0.3770.142 (Official Build) (64-bit)
macOS Mojave 10.14.6

Any progress so far?

@ghost
Copy link

ghost commented Jul 30, 2019

Same, should be about time it get's fixed by now no? It's been a problem for several years now, it seems to think it removed it (at least it claims so), but it doesn't actually do anything.. The whole new tab page is about as customizable as internet explorer 8 was though, it's the only big issue about brave that still might turn me away some day.

Version 0.66.101 Chromium: 75.0.3770.142 (Official Build) (64-bit)

@chrisB28
Copy link

Still an issue here


Brave | 0.66.101 Chromium: 75.0.3770.142 (Official Build) (64-bit)

Revision | 8d93477bd18154c67a70a67019cb8032382c55dd-refs/branch-heads/3770@{#1160}

@stingray21
Copy link

Same issue

Version 0.66.99 Chromium: 75.0.3770.100 (Official Build) (64-bit)
Linux (Arch)

@chrisB28
Copy link

chrisB28 commented Aug 1, 2019

To add to this, if I try dragging a normal (not deleted) icon to one of the icons that is meant to be removed, the page adds an exponential amount of the removed icon to the list.

Example:

  • Try to remove github.com icon from new tab page. Icon is not removed.
  • Drag google.com icon onto github icon.
  • 100s of github icons appear on dragging over original github icon.

Same stats as my comment above.

@cezaraugusto cezaraugusto assigned cezaraugusto and unassigned imptrx Aug 7, 2019
@haserbuen
Copy link

uninstalled brave until they get it together and start fixing bugs. Worse than Mozilla!

@chrisB28
Copy link

chrisB28 commented Oct 7, 2019

Resolved for me after updating today!

@alexandrestein
Copy link

I'm using version: Version 0.69.132 Chromium: 77.0.3865.90 (Build officiel) unknown (64 bits) on Arch but the bug still there for me.

@chrisB28
Copy link

chrisB28 commented Oct 7, 2019

I'm using version: Version 0.69.132 Chromium: 77.0.3865.90 (Build officiel) unknown (64 bits) on Arch but the bug still there for me.

I had one that was still doing it then I made sure the pin was deselected and it removed correctly

@alexandrestein
Copy link

Thank you @chrisB28.
It started with have a strange behavior after unpin every items and then it started to work as expected.

👌

@rebron
Copy link
Collaborator

rebron commented Oct 8, 2019

@cezaraugusto Should we close this one out? It looks like this is working now. I can't reproduce the issue with current release channel 69.x.

@alexandrestein Can you provide specific steps to reproduce? Just clicking 'x' on one of the top site tiles isn't removing the tile for you?

@alexandrestein
Copy link

@rebron I can't reproduce it. 😟
It is working normally now. 😄

I tried long time ago to manage top sites.
Because it was not working as I expected I followed this issue and I let them as is.

Some was pined and others not. I can't remember what was the exact status of the thing.

@chrisB28 announced that it was resolved.
I tried but it was still buggy.
@chrisB28 advised me to unpin site. I unpinned all sites and then weird things happened (reorder and blinking) before returning to a working state.

Some my case is not relevant, sorry about the noise.

@yzhernand
Copy link

I had to click "restore all" after removing a pinned top site still failed with the latest update, but it showed the message that the site had been removed. Now I have one top site pinned from one of the "restored" sites, and nothing new is being suggested.

Version 0.69.132 Chromium: 77.0.3865.90 (Official Build)
EndeavourOS (Arch Linux) 64-bit

@rebron
Copy link
Collaborator

rebron commented Oct 9, 2019

Closing out this issue, doesn't look like this is an issue any longer? I'll reopen if this is still an issue for @jlarne?

@yzhernand would you mind filing a new issue? That's a separate issue from what's happening here.

@rebron rebron closed this as completed Oct 9, 2019
@rebron rebron added closed/not-actionable and removed priority/P3 The next thing for us to work on. It'll ride the trains. regression labels Oct 9, 2019
@jlrnr
Copy link
Author

jlrnr commented Oct 10, 2019

It seems to be working for me as well! Thanks for the follow up!

@ghost
Copy link

ghost commented Oct 10, 2019

Odd, still doesn't work for me, and am on the latest build on an up to date windows 10

@rebron
Copy link
Collaborator

rebron commented Oct 10, 2019

@The-Clockmaker I'm testing the same on Windows 10 with the current 69.132 release and I'm able to remove top sites. We need some more info. I'm assuming you don't have any of the tiles pinned?

Clearing browser history does reset the Top Sites list. Not sure if that's something you can help us test.

@dodzh
Copy link

dodzh commented Oct 22, 2019

ive had this same issue for a while now also.
Version 0.69.135 Chromium: 77.0.3865.120 (Official Build) (64-bit)

@bsclifton
Copy link
Member

cc: @cezaraugusto on the above report by @dodzh

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

No branches or pull requests