All users were logged out of Bugzilla on October 13th, 2018

Search results app uses a white icon, rather than a dedicated nicer looking app icon

VERIFIED FIXED in Firefox OS v2.0

Status

VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: jsmith, Assigned: gasolin)

Tracking

unspecified
2.0 S6 (18july)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(blocking-b2g:2.0+, b2g-v1.4 unaffected, b2g-v2.0 verified, b2g-v2.1 verified)

Details

(Whiteboard: [systemsfe][p=1])

Attachments

(4 attachments, 1 obsolete attachment)

(Reporter)

Description

4 years ago
Created attachment 8436692 [details]
Screenshot

Build

* Device: Flame
* Version: 2.0 - b2g-inbound
* Build ID: 20140608173002
* Gaia: 12af93123c5db55212d51fe235d39f21209a1eaa
* Gecko: 7f3f35b511f8

STR

1. Go to app permissions

Expected

The search results app should have a nicer looking app icon representative of the app.

Actual

The search results app has a white icon.
(Reporter)

Updated

4 years ago
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking-]
Whiteboard: [systemsfe]
See Also: → bug 1038167
Peter, seems like we'll also need an icon here. This is for the "search results" app permissions, the search results app is the app that opens up when you focus on the rocketbar.
Flags: needinfo?(pla)
blocking-b2g: --- → 2.0?
status-b2g-v2.0: --- → affected
status-b2g-v2.1: --- → affected
Noming for blocking as we should ship with an icon for the search app.  
We would need a @1.5x @2x @2.25x as well I think for the manifest?
No longer blocks: 1035684
Flags: needinfo?(firefoxos-ux-bugzilla)

Comment 3

4 years ago
Agree with blocking call. Thanks Naoki! Peter La is the correct person for the icon and is already flagged, so I'll leave that flag set.
Flags: needinfo?(firefoxos-ux-bugzilla)

Comment 4

4 years ago
Thanks for pointing this out guys.  I was about to file a bug for it.  I'll attach the icon tonight.

Leaving NI flag on so I remember.

Comment 5

4 years ago
Created attachment 8458421 [details]
SearchResultsIcon-20140717.zip

Here is the icon for Search Results for App Permissions.
Flags: needinfo?(pla)

Comment 6

4 years ago
Created attachment 8458422 [details]
AppPermissionsIcons-Mockup.jpg

And here is the mockup of how it looks in context.
(Assignee)

Comment 7

4 years ago
Hi I've seen several app use 84/126/142/189/284 instead of @2x.png style

The provided size are 80/126/142/189/256/284 . should the 80x80 icon be 84x84?
Flags: needinfo?(pla)
(Assignee)

Comment 8

4 years ago
Created attachment 8458446 [details] [review]
pull request redirect to github

wait for icon size feedback before review
Assignee: nobody → gasolin

Comment 9

4 years ago
(In reply to Fred Lin [:gasolin] from comment #7)
> Hi I've seen several app use 84/126/142/189/284 instead of @2x.png style
> 
> The provided size are 80/126/142/189/256/284 . should the 80x80 icon be
> 84x84?

Hi Fred,

If you've seen reference to 84, then that is a mistake.  There have never been icons shipped at 84.  The sizes should be 80/126/142/189/256/284 and we always shipped them in those sizes for 2.0.
Flags: needinfo?(pla)

Comment 10

4 years ago
(In reply to Peter La from comment #9)
> (In reply to Fred Lin [:gasolin] from comment #7)
> > Hi I've seen several app use 84/126/142/189/284 instead of @2x.png style
> > 
> > The provided size are 80/126/142/189/256/284 . should the 80x80 icon be
> > 84x84?
> 
> Hi Fred,
> 
> If you've seen reference to 84, then that is a mistake.  There have never
> been icons shipped at 84.  The sizes should be 80/126/142/189/256/284 and we
> always shipped them in those sizes for 2.0.

Btw, technically there only needs to be 80/126/142/189/284.  The 256 size was just something Kevin asked for 'in case' it was needed.  Kevin, maybe you can explain the potential need for the 256 size.
Flags: needinfo?(kgrandon)
(In reply to Peter La from comment #9)
> (In reply to Fred Lin [:gasolin] from comment #7)
> > Hi I've seen several app use 84/126/142/189/284 instead of @2x.png style
> > 
> > The provided size are 80/126/142/189/256/284 . should the 80x80 icon be
> > 84x84?
> 
> Hi Fred,
> 
> If you've seen reference to 84, then that is a mistake.  There have never
> been icons shipped at 84.  The sizes should be 80/126/142/189/256/284 and we
> always shipped them in those sizes for 2.0.

I think that's wrong? 84 is expected and we support that for all of the apps. I am fairly certain we should be using 84px.
Flags: needinfo?(kgrandon) → needinfo?(pla)
The sizes we want here are as follows:

84
126
142
189
284

Comment 13

4 years ago
Hey Kevin/Fred,

You guys are actually correct.  I got it mixed up, sorry. :(  Fred, so it is actually 84/126/142/189/284 like you said, but somehow I made the mistake of changing the @1 size to 80 when providing these new icons.  This was also the case for this bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1038167

Will update with the correct size now in both bugs. :(
Flags: needinfo?(pla)

Comment 14

4 years ago
Created attachment 8458697 [details]
SearchResultsIcon-20140718.zip

Here are the Search Results icon with the proper @1 size of 84.

Sorry again for the mixup.
Attachment #8458421 - Attachment is obsolete: true
(Assignee)

Comment 15

4 years ago
Comment on attachment 8458446 [details] [review]
pull request redirect to github

updated 84x84 and run through png_recompress.sh
Attachment #8458446 - Flags: review?(kgrandon)
blocking-b2g: 2.0? → 2.0+
Comment on attachment 8458446 [details] [review]
pull request redirect to github

Looks good to me, thanks!
Attachment #8458446 - Flags: review?(kgrandon) → review+
(Assignee)

Comment 17

4 years ago
merged to gaia-master https://github.com/mozilla-b2g/gaia/commit/7104d2996b9ad65d9fff57ada454699d3348f0a5
Status: NEW → RESOLVED
Last Resolved: 4 years ago
status-b2g-v2.1: affected → fixed
Resolution: --- → FIXED
Whiteboard: [systemsfe] → [systemsfe][p=1]
Target Milestone: --- → 2.0 S6 (18july)
Gecko     https://hg.mozilla.org/mozilla-central/rev/e9cdcf646d1c
BuildID   20140720160202
Version   33.0a1
ro.build.version.incremental=110
ro.build.date=Fri Jun 27 15:57:58 CST 2014
B1TC00011230

Gaia      bf3fb88039843359d0a5759b2c0fb787abae544f
Gecko     https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/590161396a0c
BuildID   20140722160201
Version   32.0
ro.build.version.incremental=110
ro.build.date=Fri Jun 27 15:57:58 CST 2014
B1TC00011230
Status: RESOLVED → VERIFIED
status-b2g-v1.4: --- → unaffected
status-b2g-v2.0: fixed → verified
status-b2g-v2.1: fixed → verified
You need to log in before you can comment on or make changes to this bug.