659266 - Need a minimalistic Sync glyph for notifications

Status: VERIFIED FIXED

(Firefox :: Theme, defect)

Description

[Marco Bonardo [:mak]]

bug 618913 landed with a 24px Sync icon, but UX would prefer a more minimalistic icon for it, replacing the icon is trivial, so it may land on central and Aurora.
Tentatively assigning to Stephen.

Comment 1

[Stephen Horlander [:shorlander]]

Attachment: Sync Glyph for the Panel Notification - 01

Adds low key images for the Sync notification.

Comment 2

[Stephen Horlander [:shorlander]]

Attachment: Screenshot - 01

Comment 3

[Marco Bonardo [:mak]]

it's really nice.
A couple questions:
1. Is it expected that on gnomestripe we keep the current sync-24.png?
2. Why didn't you remove sync-24.png from pinstripe and winstripe? Should be unused now.

Comment 4

[Stephen Horlander [:shorlander]]

(In reply to comment #3)
> it's really nice.
> A couple questions:
> 1. Is it expected that on gnomestripe we keep the current sync-24.png?

Yes. It will work better than trying to match variable themes.

> 2. Why didn't you remove sync-24.png from pinstripe and winstripe? Should be
> unused now.

Oh yeah, I should do that :) Forgot we added that just for this.

Comment 5

[Stephen Horlander [:shorlander]]

Attachment: Sync Glyph for the Panel Notification - 02

Removed the unused sync-24.png

Comment 6

[Marco Bonardo [:mak]]

I think, for consistency, may make sense to rename the gnomestripe sync-24.png to sync-notification-24.png it may be confusing for future otherwise.
Sorry if I didn't point this out before.

Comment 7

[Marco Bonardo [:mak]]

Attachment: Sync Glyph for the Panel Notification - 03

I've renamed the icon in gnomestripe, for previous comment, moved in jar.mn according to the other themes.

Comment 8

[Marco Bonardo [:mak]]

http://hg.mozilla.org/integration/mozilla-inbound/rev/8460bb7fb589

Comment 9

[Marco Bonardo [:mak]]

http://hg.mozilla.org/mozilla-central/rev/8460bb7fb589

Comment 10

[Marco Bonardo [:mak]]

Comment on attachment 543767 [details] [diff] [review]
Sync Glyph for the Panel Notification - 03

May we get beta approval on this icon replacement?
I think the first impact of this notification is important, and we'll have that impact in Firefox 7. This change is a try to make the notification less intrusive to the user, since it's not intended to be annoying nor look like an advertisement.

Comment 11

[Marco Bonardo [:mak]]

ehr, in comment 5 I meant "we'll have that impact in Firefox 6"

Comment 12

[Asa Dotzler [:asa]]

Comment on attachment 543767 [details] [diff] [review]
Sync Glyph for the Panel Notification - 03

approval for beta. did this make aurora?

Comment 13

[Marco Bonardo [:mak]]

sure, it made Aurora.

Comment 14

[Marco Bonardo [:mak]]

http://hg.mozilla.org/releases/mozilla-beta/rev/6e78e6a73460

Comment 15

[AndreiD[QA]]

Verified on 6.0 Beta, build 2:
Mac: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:6.0) Gecko/20100101 Firefox/6.0
Windows 7: Mozilla/5.0 (Windows NT 6.1; rv:6.0) Gecko/20100101 Firefox/6.0
Windows XP: Mozilla/5.0 (Windows NT 5.1; rv:6.0) Gecko/20100101 Firefox/6.0
Linux: Mozilla/5.0 (X11; Linux i686; rv:6.0) Gecko/20100101 Firefox/6.0

Notifications to set up sync are visible (when bookmarking a website a message that promotes Sync is visible to the user)