Patryk Adamczyk [:patryk] UX Reporter
	Description • 10 years ago

Can we change the hang up iconography to be more clear, what about a handset with a slash through it?

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

I can take it, please assign me when the icon is ready :)

	Peter La
	Comment 2 • 10 years ago

Pavel, here is the set of icons. :)

Thanks!

	Peter La
	Comment 4 • 10 years ago

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

Hey Pavel,

I tested it on my Flame and for some reason it's using a lower-res image and scaling it up.  I also noticed the icon seems a little low but this is a bitmap issue on my end.  I'll revise the icons and upload a new set.

Thanks!

	Peter La
	Comment 5 • 10 years ago

Pavel, here is the updated icon set with the icon moved up slightly so that it's vertically centered.

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

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

Hey Peter,
can you check it again? :)

	Peter La
	Comment 7 • 10 years ago

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

Hi Pavel,

The positioning of the icon looks good now.  The icon is still fuzzy but so are the other action icons - Amy Lee tells me there is a flag you have to pass in when loading the builds to render the @1.5x icons on Flame?

	Peter La
	Comment 8 • 10 years ago

This is a screenshot of what it looks like for me (the fuzzy icons).

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

yep the flag is GAIA_DEV_PIXELS_PER_PX=1.5

	Etienne Segonzac (:etienne)
	Comment 10 • 10 years ago

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

Redirecting...
(but looking good!)

	Doug Sherk (:drs) (inactive)
	Comment 11 • 10 years ago

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

I think we should be updating these as well:
https://github.com/mozilla-b2g/gaia/tree/master/apps/callscreen/style/images/shared

	Doug Sherk (:drs) (inactive)
	Comment 12 • 10 years ago

Here's a screenshot of the place that the icon I referred to in comment 11 is used.

	Peter La
	Comment 13 • 10 years ago

Hi Doug - yes I noticed that too afterwards.  I'll provide the icon for Pavel before we can close this bug.

	Doug Sherk (:drs) (inactive)
	Comment 14 • 10 years ago

(In reply to Peter La from comment #13)
> Hi Doug - yes I noticed that too afterwards.  I'll provide the icon for
> Pavel before we can close this bug.

Ok, thanks. Setting need info on you for that.

	Peter La
	Comment 15 • 10 years ago

Hi Doug/Pavel,

Please use this icon for the screen referred to in Comment 12.  I've included 5 sizes but use the sizes that are needed.  I don't think we technically need the 3.375x one yet.

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

Thanks Peter :)

	Doug Sherk (:drs) (inactive)
	Comment 17 • 10 years ago

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

Let's remove the images that are at 3.375x. We can include them later if needed.

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

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

Sure :) PR updated

	Doug Sherk (:drs) (inactive)
	Comment 19 • 10 years ago

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

Looks good, thanks.

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

Thanks :)

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