Closed Bug 1913976 Opened 2 months ago Closed 2 months ago

menu_mobileBookmarks folder is missing.

Categories

(Toolkit :: Places, defect, P1)

Firefox 129
defect

Tracking

()

VERIFIED FIXED
131 Branch
Tracking Status
firefox-esr115 --- unaffected
firefox-esr128 --- unaffected
firefox129 --- wontfix
firefox130 + verified
firefox131 + verified

People

(Reporter: klcb2010, Assigned: mak)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:129.0) Gecko/20100101 Firefox/129.0

Steps to reproduce:

This folder usually appears when you log in on both mobile devices and a PC and sync them. However, it has now disappeared, and I don't know why. I haven't made any "CSS" changes or installed any extensions or plugins. The current test machine is running Windows 11-24H2-26100-1301.

Actual results:

The bookmark menu folder "menu_mobileBookmarks" in the new Firefox version 129 is no longer visible and seems to have been disabled. Its Chinese name is "移动设备上的书签" (Bookmarks on Mobile Devices). This folder usually appears when you log in on both mobile devices and a PC and sync them. However, it has now disappeared, and I don't know why. I haven't made any "CSS" changes or installed any extensions or plugins. The current test machine is running Windows 11-24H2-26100-1301.

Expected results:

This folder should usually appear when you log in on both mobile devices and a PC and sync them

The Bugbug bot thinks this bug should belong to the 'Firefox::Sync' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: Untriaged → Sync

I think this folder only appears when there are actually bookmarks in that folder. The code is here

(In reply to Mark Hammond [:markh] [:mhammond] from comment #2)

I think this folder only appears when there are actually bookmarks in that folder. The code is here

I am certain that there are bookmarks in the 'Mobile Bookmarks' folder, and this issue does not exist in Firefox version 128 and below."

This is working fine for me on Firefox 131. On your desktop device, what does about:config show for the preference browser.bookmarks.showMobileBookmarks?

(In reply to Mark Hammond [:markh] [:mhammond] from comment #4)

This is working fine for me on Firefox 131. On your desktop device, what does about:config show for the preference browser.bookmarks.showMobileBookmarks?

true

That's odd - it implies Sync agrees there are mobile bookmarks and this pref being true should be all that's necessary for it to appear. I've no idea why it's not being shown for you, but I'll adjust the component because it appears Sync is doing the right thing here.

Component: Sync → Places
Product: Firefox → Toolkit

(In reply to Mark Hammond [:markh] [:mhammond] from comment #6)

That's odd - it implies Sync agrees there are mobile bookmarks and this pref being true should be all that's necessary for it to appear. I've no idea why it's not being shown for you, but I'll adjust the component because it appears Sync is doing the right thing here.
I can confirm having the same issue (beginning around the last couple of weeks).

My Mobile Bookmarks folder which has dozens of bookmarks just stopped showing up on my desktop FF's bookmark menu. The value of 'browser.bookmarks.showMobileBookmarks' is 'true'.

Maybe a regression from https://bugzilla.mozilla.org/show_bug.cgi?id=1893068, let me check.

Flags: needinfo?(mak)
Assignee: nobody → mak
Severity: -- → S2
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(mak)
Keywords: regression
Priority: -- → P1
Regressed by: 1893068

Set release status flags based on info from the regressing bug 1893068

Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/57c9b445613e mobileBookmarks folder is missing in Bookmarks menu. r=places-reviewers,Gijs,Standard8
Attachment #9421327 - Flags: approval-mozilla-release?

release Uplift Approval Request

  • User impact if declined: no Mobile bookmarks folder in bookmarks menus
  • Code covered by automated testing: yes
  • Fix verified in Nightly: no
  • Needs manual QE test: yes
  • Steps to reproduce for manual QE testing: Sync bookmarks from mobile, check the mobile bookmarks folder appears in the main menubar Bookmarks menu, and in the Bookmarks menu button (must be customized in)
  • Risk associated with taking this patch: low
  • Explanation of risk level: small fix, the touched code is already broken, has unit test.
  • String changes made/needed: none
  • Is Android affected?: no
Flags: qe-verify+
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 131 Branch
Attachment #9421327 - Flags: approval-mozilla-release? → approval-mozilla-release+
QA Whiteboard: [qa-triaged]

Sorry and thanks for fixing this! (I guess I didn't test mobile bookmarks at all, because my test profile didn't have sync enabled...)

Verified that the Mobile Bookmarks folder is displayed, available and updated on sync, on both Firefox Nightly 131.0a1, and Firefox 130.0 build 2.
Verified with Windows 10, and Google Pixel 6 (Android 15).

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: