Syncing… in devices list should say Syncing Tabs…

RESOLVED FIXED in Firefox 57



2 years ago
2 years ago


(Reporter: rfeeley, Assigned: stevea1, Mentored)



55 Branch
Firefox 57

Firefox Tracking Flags

(firefox57 fixed)



(2 attachments)



2 years ago
"Sync Now" in Synced Tabs does not perform a full Sync but rather just refreshes the list of devices. It should therefore just say "Refresh Devices".

While refreshing it should say "Refreshing…" (and not "Syncing…")

The tooltip should not be prefixed with "Last sync:" but rather "Updated:"

Comment 1

2 years ago
A correction to the above… After learning that this feature also syncs the currently opened tabs, we can take a much less radical approach.

The only change required is to change the mid-sync state of "Syncing…" to "Syncing Tabs…"

That provides just enough context for all users to understand that a full sync is not happening.
Summary: Sync Now in devices list should say Refresh Devices → Syncing… in devices list should say Syncing Tabs…
Mentor: eoger
Keywords: good-first-bug
Priority: -- → P2

Comment 2

2 years ago
Hi, I am a beginner. Can I take it up? Also, it would be great if you could provide me with some starter points!
Flags: needinfo?(rfeeley)

Comment 4

2 years ago
Hi, I'm new as well. I'm submitting the attached patch as my first. Please advise on next steps!


2 years ago
Attachment #8901615 - Flags: review?(eoger)
Comment on attachment 8901615 [details] [diff] [review]
Change the mid-sync state of "Syncing…" to "Syncing Tabs…"

Review of attachment 8901615 [details] [diff] [review]:

Thanks for the patch, but due to how Firefox is localized, the string ID also needs to be changed. ie, probably needs to change to something like:

syncingtabs.label = Syncing Tabs…

and all references in the code to |syncing2.label| need to change to |syncingtabs.label|

Could you please make these changes and re-upload the patch for review. Ideally, you can try to follow the dev guide and push the patch to mozreview. If you change the commit message to |Bug 1390644 - Change the mid-sync state of "Syncing…" to "Syncing Tabs…". r?eoger", then when you push to mozreview it will automatically ask Edouard for review and will simplify the process of pushing it once it is accepted. Let us know if you need help with any of that!
Attachment #8901615 - Flags: review?(eoger)
Assignee: nobody → stevea1
Flags: needinfo?(rfeeley)
Comment hidden (mozreview-request)

Comment 7

2 years ago
Ok, I've made the recommended changes and pushed to mozreview.

Comment 8

2 years ago
Comment on attachment 8902217 [details]
Bug 1390644 - Change the mid-sync state of "Syncing…" to "Syncing Tabs…".

Works great, thank you for your patch!
Attachment #8902217 - Flags: review?(eoger) → review+

Comment 9

2 years ago
Pushed by
Change the mid-sync state of "Syncing…" to "Syncing Tabs…". r=eoger

Comment 10

2 years ago
Last Resolved: 2 years ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
You need to log in before you can comment on or make changes to this bug.