Pavel Ivanov [:ivanovpavel][:pivanov] UX Assignee
	Description • 10 years ago

      No description provided.

	Pavel Ivanov [:ivanovpavel][:pivanov] UX Assignee
	Comment 1 • 10 years ago

Hey Fang,

We need to check Headers images ... take a look here:
https://github.com/mozilla-b2g/gaia/tree/master/shared/style/headers/images/icons

Some of the Apps like Email use icons from header

	Fang Shih [:fang]
	Comment 2 • 10 years ago

Hi Pavel,

For Email header, icon, the back and next arrows are the only missing icon, and I already attached in bug 1023140 comment 7. Thanks!

	Pavel Ivanov [:ivanovpavel][:pivanov] UX Assignee
	Comment 3 • 10 years ago

Yep,
I work on a patch now ... I just want to check all images in header :) 

Thanks :)

	Fang Shih [:fang]
	Comment 5 • 10 years ago

Comment on attachment 8438980 [details]
patch for Gaia/master

Header icons are looking sharp now!
But noticed that all the tab icons are fuzzy now? Or it just happened on this patch only? Since this is a bug of header images. Just want to double confirm : ) Thanks!

	Pavel Ivanov [:ivanovpavel][:pivanov] UX Assignee
	Comment 6 • 10 years ago

Hey Fang,

the replay icons are part from the Headers too.
we also need for the headers following version of icons:

add@2x.png
clear@2x.png
clear@2.25x.png
edit@2x.png
options@2x.png
reply.png@2.25x.png
reply_all@2.25x.png
user@2x.png

	Pavel Ivanov [:ivanovpavel][:pivanov] UX Assignee
	Comment 7 • 10 years ago

What do you mean by `tab icons`?

	Helen Huang [:HHuang]
	Comment 8 • 10 years ago

Hi Pavel, images have been updated.

	Fang Shih [:fang]
	Comment 9 • 10 years ago

(In reply to Pavel Ivanov [:ivanovpavel] from comment #7)
> What do you mean by `tab icons`?

Sorry, I mean to say tool bar icons. 
After our offline discussed, once the bug 1023140 landed, the tool bar​ icon will be good! I have no question now :-) Thanks a lot!

	Arnau March [:arnau] ( not working in Firefox OS anymore :( )
	Comment 10 • 10 years ago

Pavel, have some comments in github :)

	Pavel Ivanov [:ivanovpavel][:pivanov] UX Assignee
	Comment 11 • 10 years ago

Done :)

	Arnau March [:arnau] ( not working in Firefox OS anymore :( )
	Comment 12 • 10 years ago

Comment on attachment 8438980 [details]
patch for Gaia/master

Thanks Pavel. LGTM!

	Pavel Ivanov [:ivanovpavel][:pivanov] UX Assignee
	Comment 13 • 10 years ago

Thanks :) 

Landed to master:
https://github.com/pivanov/gaia/tree/bug-1024295