866376 - A new settings icon for 'Enable USB Storage' (replacing current placeholder Airplane icon)

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Settings, defect)

Description

[Evelyn Hung]

Per Casey's design for new storage related settings, we need an icon for displaying 'Enable USB Storage'. Please refer the wireframe here (page 7):
https://www.dropbox.com/s/lr35b241542fmbu/Storage_v0.1.pdf
The new icon is for place A.

Comment 1

[Evelyn Hung]

nominate leo+ because it blocks a loe+ (bug 856782)

Comment 2

[Dylan Oliver [:doliver]]

Blocking a required 1.1 feature.

Comment 3

[Evelyn Hung]

Not sure who can help on this issue, so ni UX group.

Comment 4

[Evelyn Hung]

@Arthur, please help on following up this issue. Thanks.

Comment 5

[Natalya Kot [:nkot]]

Attachment: airplane mode icon for USB storage

having airplane mode icon displayed for enabling USB storage... doesn't look right, is it temporary?

Comment 6

[Stephany Wilkes]

Going to reassign from general UX to Francis since this is Leo blocking. Francis, please feel free to reassign as appropriate.

Comment 7

[Francis Djabri [:djabber]]

Hi Peter, could you help us with an icon for Enabling USB storage?

Comment 8

[Evelyn Hung]

(In reply to nkot from comment #5)
> Created attachment 748303 [details]
> airplane mode icon for USB storage
> 
> having airplane mode icon displayed for enabling USB storage... doesn't look
> right, is it temporary?

yes, so I file this issue for UX support on a correct icon.

Comment 9

[Daniel Holbert [:dholbert]]

(I nearly filed a duplicate bug about the airplane icon being used for USB storage, and then I discovered this existing bug just in time. --> Updating bug-summary to mention "airplane" to make this bug more searchable for anyone else who runs into this.)

Comment 10

[Stephany Wilkes]

Do we have the correct icon yet? This is leo+ and needs to move. Thanks!

Comment 11

[Peter La]

Attachment: Icon

Apologies for the delay.  Attached is the icon.

Comment 12

[Arthur Chen [:arthurcc][inactive after 6/30]]

Peter, thanks for the icon.
Could you help put the icon to the sprites? (the path is /apps/settings/style/images/icons_sprite.gif, icons_sprite_active.gif) And it would be great if you can also provide the 2X version of the tow sprites. Thanks!

Comment 15

[Peter La]

Attachment: New icon placed inside icons_sprite.gif and [e-mail]

Attached are two files - icons_sprite.gif and [e-mail] - with the new icon placed at the end.

Eric, can you review the @2x version and tell me if the transparency is ok?  I'm not familiar with why it's being done with transparency when the normal version has a solid grey background.  The edges look really rough to me, but this would be ok if it's only going to appear on a white background.

Thanks!

Comment 16

[Arthur Chen [:arthurcc][inactive after 6/30]]

Attachment: icons when active

Thanks for the effort, Peter! For better performance, we avoided to use transparent images for the icons. That's why we have another image sprite with a background color for active items as the attachment shows.

It would be appreciated if you can provide the active icon sprite or the source file of the sprite with transparent background, so that we can generate corresponding sprites. Also, I've noticed that the 2x version of the display icon (the sun symbol) is a little bit rotated compared with the normal version.

Comment 17

[Eric Pang [:epang] UX]

Attachment: Setting Sprites

Hi Arthur, here are sprites for settings in both normal & action states (in normal and @2x sizes).

When in the folders on gaia we noticed that the @2x version had a transparent background, is this needed or is what I'm providing correct?  Please let me know if a transparent version or anything else is needed!  Thanks!

Comment 18

[Arthur Chen [:arthurcc][inactive after 6/30]]

Attachment: Link to https://github.com/mozilla-b2g/gaia/pull/10450

Eric, thanks for helping! The transparent background of the original 2x version file is wrong.

Evelyn, please help review the change, thanks!

Comment 19

[Evelyn Hung]

Comment on attachment 763957 [details]
Link to https://github.com/mozilla-b2g/gaia/pull/10450

r=me, thanks for your follow-up work! :)

Comment 20

[Arthur Chen [:arthurcc][inactive after 6/30]]

Thanks for reviewing, Evelyn. 

master: https://github.com/mozilla-b2g/gaia/commit/05c03cbf2a25e417116665273aa8ee809c48e3fc

Comment 21

[John Ford [:jhford] CET/CEST Berlin Time]

I was not able to uplift this bug to v1-train.  If this bug has dependencies which are not marked in this bug, please comment on this bug.  If this bug depends on patches that aren't approved for v1-train, we need to re-evaluate the approval.  Otherwise, if this is just a merge conflict, you might be able to resolve it with:

  git checkout v1-train
  git cherry-pick -x -m1 05c03cbf2a25e417116665273aa8ee809c48e3fc
  <RESOLVE MERGE CONFLICTS>
  git commit

Comment 22

[Arthur Chen [:arthurcc][inactive after 6/30]]

v1-train: d8d189b6a23adf1b492a0c325ab8f3fdbfbf7ad4

Comment 23

[John Ford [:jhford] CET/CEST Berlin Time]

1.1hd: d8d189b6a23adf1b492a0c325ab8f3fdbfbf7ad4

Comment 24

[hyuna.cho]

In v1-train, is it ok to use rem instead of px?

#menuItem-battery::before {
  background-position: -120px -90px;
}

#menuItem-enableStorage::before {
  background-position: -6rem -15rem;
}

#menuItem-applicationStorage::before {
  background-position: 0 -120px;
}

Comment 25

[Arthur Chen [:arthurcc][inactive after 6/30]]

Based on the font size configuration of the root element in v1-train, 1 rem equals 10 pixel. It is okay while the code looks inconsistent. I'll refine it in an appropriate patch.