Closed
Bug 866376
Opened 12 years ago
Closed 12 years ago
A new settings icon for 'Enable USB Storage' (replacing current placeholder Airplane icon)
Categories
(Firefox OS Graveyard :: Gaia::Settings, defect)
Tracking
(blocking-b2g:leo+, b2g18 fixed, b2g-v1.1hd fixed)
People
(Reporter: jj.evelyn, Assigned: arthurcc)
References
Details
(Whiteboard: [TD-41730], TaipeiWW, NeedsReview)
Attachments
(6 files)
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.
Reporter | ||
Comment 1•12 years ago
|
||
nominate leo+ because it blocks a loe+ (bug 856782)
Blocks: 856782
blocking-b2g: --- → leo?
Reporter | ||
Comment 3•12 years ago
|
||
Not sure who can help on this issue, so ni UX group.
Assignee: padamczyk → nobody
Flags: needinfo?(firefoxos-ux-bugzilla)
Reporter | ||
Comment 4•12 years ago
|
||
@Arthur, please help on following up this issue. Thanks.
Comment 5•12 years ago
|
||
having airplane mode icon displayed for enabling USB storage... doesn't look right, is it temporary?
Comment 6•12 years ago
|
||
Going to reassign from general UX to Francis since this is Leo blocking. Francis, please feel free to reassign as appropriate.
Flags: needinfo?(firefoxos-ux-bugzilla)
Updated•12 years ago
|
Flags: needinfo?(fdjabri)
Comment 7•12 years ago
|
||
Hi Peter, could you help us with an icon for Enabling USB storage?
Flags: needinfo?(fdjabri) → needinfo?(pla)
Reporter | ||
Comment 8•12 years ago
|
||
(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.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → arthur.chen
Updated•12 years ago
|
OS: Mac OS X → Gonk (Firefox OS)
Hardware: x86 → ARM
Comment 9•12 years ago
|
||
(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.)
Summary: A new settings icon for 'Enable USB Storage'. → A new settings icon for 'Enable USB Storage' (replacing current placeholder Airplane icon)
Comment 10•12 years ago
|
||
Do we have the correct icon yet? This is leo+ and needs to move. Thanks!
Assignee | ||
Comment 12•12 years ago
|
||
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!
Flags: needinfo?(pla)
Updated•12 years ago
|
Whiteboard: [TD-41730]
Target Milestone: --- → 1.1 QE3 (24jun)
Comment 15•12 years ago
|
||
Attached are two files - icons_sprite.gif and icons_sprite@2x.gif - 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!
Attachment #763272 -
Flags: feedback?(epang)
Flags: needinfo?(pla)
Assignee | ||
Comment 16•12 years ago
|
||
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.
Flags: needinfo?(pla)
Comment 17•12 years ago
|
||
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!
Flags: needinfo?(pla)
Assignee | ||
Comment 18•12 years ago
|
||
Eric, thanks for helping! The transparent background of the original 2x version file is wrong.
Evelyn, please help review the change, thanks!
Attachment #763957 -
Flags: review?(ehung)
Updated•12 years ago
|
Whiteboard: [TD-41730] → [TD-41730], TaipeiWW, NeedsReview
Reporter | ||
Comment 19•12 years ago
|
||
Comment on attachment 763957 [details]
Link to https://github.com/mozilla-b2g/gaia/pull/10450
r=me, thanks for your follow-up work! :)
Attachment #763957 -
Flags: review?(ehung) → review+
Assignee | ||
Comment 20•12 years ago
|
||
Thanks for reviewing, Evelyn.
master: https://github.com/mozilla-b2g/gaia/commit/05c03cbf2a25e417116665273aa8ee809c48e3fc
Comment 21•12 years ago
|
||
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
Flags: needinfo?(arthur.chen)
Assignee | ||
Comment 22•12 years ago
|
||
v1-train: d8d189b6a23adf1b492a0c325ab8f3fdbfbf7ad4
Flags: needinfo?(arthur.chen)
Comment 23•12 years ago
|
||
1.1hd: d8d189b6a23adf1b492a0c325ab8f3fdbfbf7ad4
status-b2g-v1.1hd:
--- → fixed
Comment 24•12 years ago
|
||
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;
}
Assignee | ||
Comment 25•12 years ago
|
||
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.
Updated•12 years ago
|
Attachment #763272 -
Flags: feedback?(epang)
You need to log in
before you can comment on or make changes to this bug.
Description
•