Peter La Reporter
	Description • 10 years ago

Send Kevin the updated assets for replacement into the build at the following 3-column sizes for the perfect crispness:

84, 126, 142, 189 (and maybe 284 for HD)

Also send 256 just in case.

	Peter La Reporter
	Comment 2 • 10 years ago

Hi Kevin,

Attached are the new set of App icons at the following sizes:

@1 > 84 (HVGA)
@1.5 > 126 (WVGA/FWVGA)
@2 > 142 (qHD) - this should actually be called @1.6875x but I'm going with the convention that was already established
@2.25 > 189 (720p)
@3.375 > 284 (full HD)


I've also attached a 256x256 generic size in case you need it.

Please flag me for uireview.  Thanks!

	Kevin Grandon :kgrandon Assignee
	Comment 3 • 10 years ago

Thanks. Icons for apps should probably just be in folders per app, and also labeled by their size, so icon_128.png, etc. I'll go ahead and move these into the proper locations though.

	Kevin Grandon :kgrandon Assignee
	Comment 4 • 10 years ago

I am not a sed master, but i can write a quick script to rename all of the icons in a format that I want :)

	Kevin Grandon :kgrandon Assignee
	Comment 5 • 10 years ago

Work in progress

	Kevin Grandon :kgrandon Assignee
	Comment 6 • 10 years ago

This is done, but does not yet resolve the "fuzzy" icon issue in the vertical homescreen. The problem that's causing that is likely a miss in our size selection pick. Let's solve that in bug 1014364.

	Kevin Grandon :kgrandon Assignee
	Comment 7 • 10 years ago

Comment on attachment 8428199 [details]
Github pull request

It seems like this is easier to get in as a single patch instead of smaller ones. Vivien - could you do a review for this cross-gaia change? Thanks!

	Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)
	Comment 8 • 10 years ago

I think you also need to fix a few places in the Gaia were the size 60 is hardcoded ?

Like apps/system/js/app_window.js, or the old homescreen code.

	Kevin Grandon :kgrandon Assignee
	Comment 9 • 10 years ago

Comment on attachment 8428199 [details]
Github pull request

Ugh yeah.. seems like this will be more work than I was hoping :)

Let me run through the hard-coded places and will re-open this for review. Thanks!

	Peter La Reporter
	Comment 10 • 10 years ago

Hey Kevin,

Sorry to do this - I found a few icons I wanted to tweak after the weekend.  I made tweaks to the following icons and re-exported the entire set.

> Settings
> Cost Control (Usage)
> Gallery
> Wallpaper App (not sure if this is still used?)

Sorry for the inconvenience.

Peter

	Kevin Grandon :kgrandon Assignee
	Comment 11 • 10 years ago

Comment on attachment 8428199 [details]
Github pull request

Hey Vivien - looks like we are pretty smart and don't use a hardcoded 60 value in the UI. See: https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/app_window.js#L1429

I would like to open this up for review, but I don't think we should land until we turn on the homescreen by default. Thanks!

	Kevin Grandon :kgrandon Assignee
	Comment 13 • 10 years ago

Landed: https://github.com/mozilla-b2g/gaia/commit/af744242d331fd2da873e3db8a86ab9ae9ab511a

	Kevin Grandon :kgrandon Assignee
	Comment 14 • 10 years ago

Mass modify - set status-b2g-v2.0 fixed for fixed bugs under vertical homescreen dependency tree.