1022647 - Web app icons sizes for Vertical homescreen

Status: RESOLVED WONTFIX

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

Description

[Amir Nissim (:amirn)]

The icons in the vertical homescreen are bigger than before.
E.me needs to provide bigger icons for web results to be used when saving to homescreen.

Comment 1

[Amir Nissim (:amirn)]

Patryk, can you please confirm what sizes are used in the vertical homescreen? (for both 3 and 4 columns)

Thanks

Comment 2

[Cristian Rodriguez (:crdlc)]

I think that you should call to gridMaxIconSize to know the size

https://github.com/mozilla-b2g/gaia/blob/master/shared/elements/gaia_grid/js/grid_layout.js#L102

Comment 3

[Peter La]

Hey guys,

The icon sizes for the new vertical homescreen are as follows.

    320x480   HVGA  3.5”   165ppi   Most in market devices
        Large 84px
        Small 64px

    480x800   WVGA  4.0”   233ppi   Helix (Huawei) or ZTE Open C
        Large 126px
        Small 96px

    480x854   FWVGA 4.5”   218ppi   Firefox Flame, our reference phone
        Large 126px
        Small 96px

    540x960   qHD   4.5”   245ppi   Alcatel Onetouch Fire S
        Large 142px
        Small 108px

    720x1280  720p  4.7”   312ppi   LG - Madai
        Large 189px
        Small 144px

    1080x1920 1080p 5”?    440ppi   Still to be determined
        Large 284px
        Small 216px

Comment 4

[Cristian Rodriguez (:crdlc)]

This method will give you this info

(In reply to Cristian Rodriguez (:crdlc) from comment #2)
> I think that you should call to gridMaxIconSize to know the size
> 
> https://github.com/mozilla-b2g/gaia/blob/master/shared/elements/gaia_grid/js/
> grid_layout.js#L102

Comment 5

[Jason Smith [:jsmith]]

Moving to VH next since we won't stop ship of the release with this issue. If it's implemented in time, then we'll get approval.

Comment 6

[Kevin Grandon :kgrandon]

Mass update: Resolve/Wontfix all existing collections bugs as this feature is now removed.

Please re-open or file a new bug if you feel that this bug still exists in master.