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

      No description provided.

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

Hey Patryk, 

the icons "<", "play", "pause", ">" are old ... can I use the same icons from Music App or I need to ping someone for the new icons?

	Patryk Adamczyk [:patryk] UX
	Comment 2 • 10 years ago

Przemek can you please provide Pavel with the latest.

	Przemek Abratowski [:przemek] UX
	Comment 3 • 10 years ago

Is this for 1.4 or 2.0?
Amy is re-working the lockscreen for 2.0 right?

	Amy Lee [:amylee] UX
	Comment 4 • 10 years ago

(In reply to Przemek Abratowski [:przemek] UX from comment #3)
> Is this for 1.4 or 2.0?
> Amy is re-working the lockscreen for 2.0 right?

Hi Przemek, 

The lockscreen is for 2.0. We are still using the same icons for the music player though.

	Przemek Abratowski [:przemek] UX
	Comment 5 • 10 years ago

I just talked to Amy, she will provide the media controls.

	Amy Lee [:amylee] UX
	Comment 6 • 10 years ago

Hi Pavel, 

Here are the music player controls. I am wondering if the reason they are fuzzy is because they have been resized incorrectly. The flame is suppose to use @1.5 scaled graphics but the current screenshot shows the icons at a size that is somewhere in between @1.5 and @2 scale. I've attached the @1.5 scaled graphics (these are a bit smaller than the current screenshot).

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

Thanks Amy :)
actually the @1.5 icons missing from system/media_playback I will create a patch soon for them

	Pavel Ivanov [:ivanovpavel][:pivanov] UX Reporter
	Comment 8 • 10 years ago

Hey Amy ... I just realize that we use bigger icons for the lockscreen ... you can see Bug 891024 ... can you provide me the bigger icons in all variation @1, @1.5 etc. and I will update them too.

P.S. for the normal icons I use the icons from Music App because current icons are old (with shadow)

	Amy Lee [:amylee] UX
	Comment 9 • 10 years ago

Hi Pavel, 

Okay that makes more sense now. I was wondering why in the screencap the icons were bigger than what we have in the icon sheets. I've attached the icons in the larger size based on bug 891024 @1, @1.5, @2. Let me know if you need anything else! p.s what version are we releasing this for? There is also a visual refresh of the lockscreen for 2.0 that also has the music player in it, Bug 950884. Just want to make sure we use the larger icons in there as well.

	Amy Lee [:amylee] UX
	Comment 10 • 10 years ago

Here are the big version of the music player icons for lockscreen. Let me know if you need anything else!

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

Hey guys,
who can help me to test this on real device?

	Amy Lee [:amylee] UX
	Comment 12 • 10 years ago

Hi Pavel, 

This is what I'm seeing on the Flame after flashing the patch.

	Amy Lee [:amylee] UX
	Comment 13 • 10 years ago

Screenshot 2

	Pavel Ivanov [:ivanovpavel][:pivanov] UX Reporter
	Comment 14 • 10 years ago

looks good, right?

	Amy Lee [:amylee] UX
	Comment 16 • 10 years ago

Hi Pavel, 

I compared my screen (see comment 15) to the original one you attached and the icons look the same.

	Pavel Ivanov [:ivanovpavel][:pivanov] UX Reporter
	Comment 17 • 10 years ago

Hey Amy, 
as we talk on IRC ... I find the problem and now is OK I think (you can try to flash your device again but don't forget to get latest of my patch). I also find that we miss few more variation of the icons:
we need @2x.png & @2.25x.png versions for:	
icon-next	
icon-pause	
icon-play	
icon-previous

	Amy Lee [:amylee] UX
	Comment 18 • 10 years ago

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

Hi Pavel, 

The icons looks sharp now. Thanks!

	Amy Lee [:amylee] UX
	Comment 19 • 10 years ago

Hi Pavel, 

Here are the controls. Cheers

	Patryk Adamczyk [:patryk] UX
	Comment 20 • 10 years ago

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

Amy will review this.

	Jim Porter (:squib)
	Comment 21 • 10 years ago

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

rs=me

	Pavel Ivanov [:ivanovpavel][:pivanov] UX Reporter
	Comment 22 • 10 years ago

Thanks all,

Landed to master:
https://github.com/mozilla-b2g/gaia/commit/29a19e61a3b5fae7d257fdc0a56fbad2055576dc