910324 - [e.me][feature] Equalize size of Evme and Grid icon’s

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Everything.me, defect)

Comment 1

[Ran Ben Aharon [:ranbena] EverythingMe]

Additional information:
In v1.2 Evme results are a more integral part of the homescreen. It makes sense that grid icons and Evme result icons have the same look and feel.

Initially we'd like to make them the same size. Probably going to resize the Evme results.

Comment 3

[Ran Ben Aharon [:ranbena] EverythingMe]

Evyatar, the psd resides here https://www.dropbox.com/s/z4vjy4ypfed9dec/A01_v1.2%20%28Balanced%29.psd
Notice the icon sizes and spacings.

Comment 4

[Evyatar 'Tron' Amitay (everything.me)]

on branch *results-design*

Commit: 6b0943b031d9b911e541d17f1a8fba94f4136015
https://github.com/EverythingMe/gaia/commit/6b0943b031d9b911e541d17f1a8fba94f4136015


NOTE this will not fix the apps ABOVE the line (pinned apps) since this was fixed in a different ticket (911772).

Comment 6

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

To note, we have 3 sets of sizes for icons which are dictated by part of our build script for gaia (which in turn depends on the device).  Is it possible to have various images depending on screen resolution/size for e.me so that the icons won't get pixelated/stretched, etc?

Comment 7

[Ran Ben Aharon [:ranbena] EverythingMe]

Sure. Every Collection icon has a manifest "collection.manifest" which defines icon sizes and corresponding image paths, just like apps do.

This will be addressed in Bug 910338

Comment 8

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Ah.  Thanks!  I missed seeing that bug.

Comment 9

[Evyatar 'Tron' Amitay (everything.me)]

Attachment: Patch - redirect to github PR

Comment 10

[Cristian Rodriguez (:crdlc)]

Comment on attachment 811971 [details]
Patch - redirect to github PR

I added some comments on github for homescreen part. Please Ran review evme code. Thx

Comment 11

[Ran Ben Aharon [:ranbena] EverythingMe]

The semi-transparent circular background layer is part of the rendered image.
This creates a few design quirks in edit mode:
1. The background pulsates and overlaps the icon's circular border.
2. When dragging an an app icon the rendered image shrinks, exposing a gap between the background and the border.
3. The gap gets bigger when dragging an app over a Collection icon.

Imo it's a quirk. Might be by design.

Comment 12

[Cristian Rodriguez (:crdlc)]

I was testing this patch and my collection icons are not updated after drop an application. Am I doing something wrong?

Comment 13

[Cristian Rodriguez (:crdlc)]

Inside a collection, if I enter in edit mode the ace to delete apps seem to be placed wrongly according to grid

Comment 14

[Ran Ben Aharon [:ranbena] EverythingMe]

Cristian, the app drop does work - only you might not see the icon updating because it's added last. Open the collection - you'll see the app there.

This will change in Bug 921092.

Comment 15

[Cristian Rodriguez (:crdlc)]

Yes, icons are saved perfectly I know. I expected that the icon changes, alright, I can see the other bug ;) Thx

Comment 16

[Ran Ben Aharon [:ranbena] EverythingMe]

Evyatar, I agree with Cristian that the X (delete button) is now mispositioned in Collection results. I assume it's because of icon size changes?

Comment 17

[Evyatar 'Tron' Amitay (everything.me)]

Addressed and pushed both issues - Remove button position and the dark underlay (moved it to be the background of the border element).

Comment 18

[Evyatar 'Tron' Amitay (everything.me)]

landed in master
b8cc3a7dfe6a7a78f48533e1712c7d5c0479f074

https://github.com/mozilla-b2g/gaia/commit/b8cc3a7dfe6a7a78f48533e1712c7d5c0479f074

Comment 19

[Evyatar 'Tron' Amitay (everything.me)]

removing koi? since e.me 1.2 features are delayed for future version