Closed Bug 1320521 Opened 8 years ago Closed 7 years ago

Duplicate bookmarks create duplicate shortcuts in the Top Sites Panel

Categories

(Firefox for Android Graveyard :: Awesomescreen, defect, P2)

defect

Tracking

(fennec+, firefox50 affected, firefox51 affected, firefox52 wontfix, firefox53 affected, firefox54 affected)

RESOLVED DUPLICATE of bug 1382332
Tracking Status
fennec + ---
firefox50 --- affected
firefox51 --- affected
firefox52 --- wontfix
firefox53 --- affected
firefox54 --- affected

People

(Reporter: rolyc5, Unassigned, Mentored, NeedInfo)

Details

(Keywords: good-first-bug)

Attachments

(4 files)

When there are multiple bookmarks to the same site, Firefox for Android will create duplicate shortcut tiles in the Top Sites panel of the home screen. Attached is a screenshot of Aurora v52 running on a Nexus 7 (2012). The expected behavior is not to have duplicate tiles to the same site. As you can see, there are 3 tiles pointing to Ars Technica. I had 3 bookmarks pointing to it: 1 in Bookmarks Menu, 1 in Other Bookmarks, and 1 in Mobile Bookmarks. They all pointed to the same URL of http://arstechnica.com/ When I deleted all but the Bookmarks Menu bookmark in the desktop bookmark manager, forced the 2 devices to Sync Now, and swiped away Aurora from the multitasking overview, the duplicate tiles went away. Devices: Firefox Beta v51.0b3 on macOS 10.12.1 Firefox Aurora v52.0a2 updated daily on Nexus 7 (2012) running a custom ROM. Android v6.0.1
Attached image Screenshot_20161205-222205.jpg (deleted) —
Thank you for report! Tested with Asus ZenPad 8 (Android 6.0.1) and the issue is reproducible on all branches. I will change the status to New and set the flags accordingly.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Version: 52 Branch → Trunk
Too late for firefox 52, mass-wontfix.
This is the behavior of this bug in Firefox 54 Beta 1. And Switch to tab is shown for one URL (https://people-mozilla.org/~fqueze2/webrtc/).
tracking-fennec: --- → ?
[triage] P2, and maybe even P3 if we will have clear timeline for activity stream landing in the near future.
tracking-fennec: ? → +
Keywords: good-first-bug
Priority: -- → P2
We might just need to deduplicate URLs when querying the frecency based topsites from the combined table, i.e. at: https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/db/BrowserProvider.java#1071 Note: the same URL could conceivably be pinned multiple times - that means we can't deduplicate later in getTopSites(), we need to do it as early as possible, i.e. when fetching the pure topsites (before pinned/suggested sites are added). IIRC we use the same query for AS, so this is probably independent of AS being enabled.
Mentor: ahunt
Hi, may I please work on this bug?
Flags: needinfo?(andrzej)
If I can work on it, what URLs are considered duplicates? Are they considered duplicates when they are exact copies or a subpages. For example, there can be facebook.com and facebook.com/home in the top sites. In this case, should they both be included? If not, which one should be displayed?
I will be fixing this as part of bug 1382332. I'm sorry no one got back to you, Varun and that now the issue will be fixed. If you would like to have another bug to work on, please add a needinfo flag for me and respond to this message.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: