Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

[Bug] Cannot delete multiple frecency top sites #14538

Closed
liuche opened this issue Aug 31, 2020 · 4 comments
Closed

[Bug] Cannot delete multiple frecency top sites #14538

liuche opened this issue Aug 31, 2020 · 4 comments
Labels
🐞 bug Crashes, Something isn't working, .. Feature:Shortcuts Top Sites/Topsites on the Firefox home page

Comments

@liuche
Copy link
Contributor

liuche commented Aug 31, 2020

Steps to reproduce

Have top sites turned on in settings
Visit some websites
Delete multiple auto-populated sites from top sites

Expected behavior

sites disappear

Actual behavior

after the first 1 or 2, sites stop getting removed even when selecting "delete from history"

Device information

  • Android device: pixel 3
  • Fenix version: Nightly 8/31

┆Issue is synchronized with this Jira Task

@liuche liuche added 🐞 bug Crashes, Something isn't working, .. Feature:Shortcuts Top Sites/Topsites on the Firefox home page labels Aug 31, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Aug 31, 2020
@liuche
Copy link
Contributor Author

liuche commented Aug 31, 2020

@LaurentiuApahideanSV i saw you QA-ed #10342 did you see this problem? I also noticed that it wasn't listed in the QA acceptance criteria so maybe that was why.

@liuche liuche removed the needs:triage Issue needs triage label Aug 31, 2020
@gabrielluong
Copy link
Member

This seems related to #14230. When we remove a frecent site, it seems like the top sites are updated to some cached version where it might appear that you have not removed the frecent site. If you try to remove it again, it ends up doing nothing because it's actually already gone.

@LaurentiuApahideanSV
Copy link

LaurentiuApahideanSV commented Sep 1, 2020

@liuche Yes, I have encountered this issue on Firefox Preview Nightly 9/1 when manually adding the same site to top sites multiple times, see issue #11188 for further information. The issue can also be reproduced by populating the top sites section with pages of the same website, for example populating the top sites section with multiple articles from https://edition.cnn.com/ and then trying to delete them.

@gabrielluong
Copy link
Member

This should be fixed with #15126.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, .. Feature:Shortcuts Top Sites/Topsites on the Firefox home page
Projects
None yet
Development

No branches or pull requests

3 participants