Closed Bug 1061330 Opened 10 years ago Closed 8 years ago

Breakdown: Implement "Synced Tabs" button and subview in Menu Panel

Categories

(Firefox :: Sync, defect, P2)

defect
Points:
3

Tracking

()

RESOLVED FIXED

People

(Reporter: MarcoM, Unassigned)

References

Details

      No description provided.
Flags: qe-verify-
Flags: firefox-backlog+
Points: --- → 3
Ryan, I have some questions...

I would expect the "Sync Now" item to cause the panel to hide while syncing. Is that assumption correct or not?

In the case where the icon is in the toolbar, do we want to animate the icon as we do now? (and if the panel is hidden when you click "Sync Now", does that mean we want to animate the icon in the panel, too?)

I'm assuming this replaces/reuses the current customizable button that just does the same as "Sync Now" ?

Are there assets for this design yet? It looks like we need panel and toolbar size icons for Linux, WinXP/aero/8, OS X (normal and yosemite) as well as lightweight themes.
Flags: needinfo?(rfeeley)
> I would expect the "Sync Now" item to cause the panel to 
> hide while syncing. Is that assumption correct or not?

Correct!

> In the case where the icon is in the toolbar, do we want
> to animate the icon as we do now? (and if the panel is 
> hidden when you click "Sync Now", does that mean we want 
> to animate the icon in the panel, too?)

Good idea! We can retain existing behaviour for the Sync button (when it is in the toolbar), but I don't recommend creating an animated version of the Synced Tabs icon.

To recap on the icon, there are now two sizes. When in the menu, and in customize mode, it is the new Synced Tab icon. And due to space constraints, when in the upper and lower toolbar, we will use the original Sync icon (which will animate when syncing).

As far as assets, I have a PSD but can export PNGs. I will work with Shorlander on getting you the assets ASAP in in the right places.
Flags: needinfo?(rfeeley)
Flags: firefox-backlog+
No longer depends on: 1201326
Flags: firefox-backlog+
Priority: -- → P2
This breakdown is done and initial feature work has landed.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.