948195 - Marketplace is incorrectly listed in the Games bubble in the Search screen

Status: RESOLVED FIXED

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

Description

[krupa raj[:krupa]]

Attachment: Marketplace listed under Games

OS version: 1.3.0.0-prerelease
Build identifier: 20131208063021
Device used in test: Fugu

steps to reproduce:
1. Start the phone such that the Search screen loads with the apps organized by functionality bubbles.
2. Notice the Games bubble

observed behavior:
Marketplace is listed under Games when it should be listed under settings

NOTE: I checked on my inari (build:20131118040206) where marketplace is listed under settings. See

Comment 1

[krupa raj[:krupa]]

Attachment: Marketplace correctly listed under Settings

Comment 2

[Yosef Waysman]

Hi Krupa

I don't agree that the Marketplace doesn't belong to the Games category. On every ecosystem, users are directed to the marketplace to find games to play. The marketplace has games - many of the most popular apps are games, and Games is the first category on the list (and probably one of the most popular of them). Therefore, I don't think any change is necessary.

Comment 3

[Jason Smith [:jsmith]]

(In reply to Yosef Waysman from comment #2)
> Hi Krupa
> 
> I don't agree that the Marketplace doesn't belong to the Games category. On
> every ecosystem, users are directed to the marketplace to find games to
> play. The marketplace has games - many of the most popular apps are games,
> and Games is the first category on the list (and probably one of the most
> popular of them). Therefore, I don't think any change is necessary.

I disagree with this. Marketplace is a not a game, so that doesn't make sense to include it as part of a games collection. It's a place to find apps, which can include gaming apps.

Bill - Can you weigh in on what you want to do here?

Comment 4

[Bill Maggs (bmaggs)]

Yosef, 

I agree with Jason and Krupa and here's why. Lots of folks on like Ran, Wil, Krupa, and others have done a lot of work so that Marketplace apps and the destination itself are discoverable for that part of a users intention to find permanent apps from Home Screen search, and I think we have a good, understandable result for exposing the cue to go the Marketplace that serves the user well. I don't think putting the Marketplace icon in any of the instant apps Smart Folders is consistent with that approach. So please, yes, remove Marketplace icon from the bubbles and the folders. 

I think a lot of us really want to merge the ideas of instant or cloud apps delivered by e.me and Marketplace apps; for a variety of reasons we are not there yet.  So it is confusing to have the Marketplace icon show up with the cloud apps because it isn't one itself, and takes people to a download experience rather than an instant apps experience. You're right that there are games in the Marketplace, but for category searches like that Ran and you have been nice enough to surface the  category labels on the Marketplace icon (see attachment for Download reference if you search for reference). BTW in the current 

BTW, on my 1.3 Keon (build id 20131210025136) the Smart Folders do not show up at all.

Comment 5

[Bill Maggs (bmaggs)]

Attachment: 2013-12-13-00-10-05.png Download reference

Comment 6

[krupa raj[:krupa]]

I agree with everything Jason and Bill have added in the above comments.

Comment 7

[Yosef Waysman]

Ok. If you think it should be removed we won't insist.
Amir - please remove the Marketplace from the Games smart collection.

Comment 8

[Amir Nissim (:amirn)]

OK, so we need to decide on 3 apps that will show up in the initial icon and create new assets.
(ref. https://github.com/mozilla-b2g/gaia/tree/master/apps/homescreen/collections/games)

The initial icon will be updated on the first launch of the collection (assuming the device is online).
Until then, the initial icon will be displayed.

Comment 9

[Yosef Waysman]

Ok. Let's use the icons of the following apps for the initial icon:
1. Free Games
2. Gem Adventure
3. Galacticans 2

Fade - please make sure the icon is created and passed to Amir.

Comment 10

[Peter Dolanjski [:pdol]]

PM retriaged this and we agree with the blocking status.

Comment 11

[Ran Ben Aharon [:ranbena] EverythingMe]

(Amir, we can create this icon on our own)

Comment 12

[Amir Nissim (:amirn)]

Attachment: Github Pull Request

Comment 13

[Ran Ben Aharon [:ranbena] EverythingMe]

Comment on attachment 8362474 [details] [review]
Github Pull Request

Great

Comment 14

[Amir Nissim (:amirn)]

landed in master: https://github.com/mozilla-b2g/gaia/commit/085a578

Comment 15

[Amir Nissim (:amirn)]

landed in v1.3: https://github.com/mozilla-b2g/gaia/commit/e16ca4feb86ac8270dff9f31b4f9d169855cd294

had some trouble when cherry-picking
bad commit: https://github.com/mozilla-b2g/gaia/commit/c93ee10134391a27f469f073f42809cb9bcbcacb
reverted here: https://github.com/mozilla-b2g/gaia/commit/c478c1f690e0eefb30242383aff2b6e50c4bdba8

Comment 16

[Gabriele Svelto [:gsvelto]]

Please when committing new or modified PNG files make sure they have been properly recompressed with the script provided under tools/png_recompress.sh. Note that to run the script you will need the optipng and advpng tools which are usually provided by the optipng and advancecomp packages on most Linux distributions.

Here's an example of how to run it and its output using the images committed in this bug:

$ ./tools/png_recompress.sh -v apps/homescreen/collections/games/icon.png \
                               apps/homescreen/collections/games/[e-mail] \
                               apps/homescreen/collections/games/[e-mail]

apps/homescreen/collections/games/icon.png 4875 3814 0.782
apps/homescreen/collections/games/[e-mail] 7840 6640 0.846
apps/homescreen/collections/games/[e-mail] 11960 10596 0.885

Number of files processed: 3
Total size of the files prior to recompression: 24675
Total size of the files after recompression: 21050
Compression ratio: 0.853

Comment 17

[Amir Nissim (:amirn)]

followup - compressed PNG:
landed in master: https://github.com/mozilla-b2g/gaia/commit/261e06dc07470d50a47d37e0104af7c7099cc803
landed in v1.3: https://github.com/mozilla-b2g/gaia/commit/99aca97f351fcd3b07dd0b47d27627899c3bcc42

Gabriel - I was not aware this is required, sorry.
Perhaps adding a commit hook when adding PNG's is a good way to alert developers in the future.

Thanks.

Comment 18

[Gabriele Svelto [:gsvelto]]

(In reply to Amir Nissim (Everything.me) from comment #17)
> Gabriel - I was not aware this is required, sorry.

It's not required, we're just trying to enforce it since we landed bug 959659.

> Perhaps adding a commit hook when adding PNG's is a good way to alert
> developers in the future.

I'm trying to implement something along the lines, I just didn't have enough time yet. Thanks for having updated the images!