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)
Firefox for Android Graveyard
Awesomescreen
Tracking
(fennec+, firefox50 affected, firefox51 affected, firefox52 wontfix, firefox53 affected, firefox54 affected)
RESOLVED
DUPLICATE
of bug 1382332
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
Updated•8 years ago
|
Blocks: as-android-newtab
Comment 1•8 years ago
|
||
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.
Updated•8 years ago
|
Status: UNCONFIRMED → NEW
status-firefox50:
--- → affected
status-firefox51:
--- → affected
status-firefox52:
--- → affected
status-firefox53:
--- → affected
Ever confirmed: true
Version: 52 Branch → Trunk
status-firefox54:
--- → affected
Comment 2•8 years ago
|
||
Too late for firefox 52, mass-wontfix.
Comment 3•8 years ago
|
||
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/).
Updated•8 years ago
|
tracking-fennec: --- → ?
Comment 4•8 years ago
|
||
[triage]
P2, and maybe even P3 if we will have clear timeline for activity stream landing in the near future.
Comment 5•8 years ago
|
||
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
Comment 6•7 years ago
|
||
Hi, may I please work on this bug?
Updated•7 years ago
|
Flags: needinfo?(andrzej)
Comment hidden (mozreview-request) |
Comment 8•7 years ago
|
||
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?
Updated•7 years ago
|
No longer blocks: as-android-newtab
Comment 9•7 years ago
|
||
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
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•