Closed
Bug 1443631
Opened 7 years ago
Closed 6 years ago
Devices in Send Tab needs latest icons
Categories
(Firefox for iOS :: Sync, defect, P2)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
fxios | 12.0 | --- |
People
(Reporter: rfeeley, Assigned: garvan)
References
(Blocks 1 open bug)
Details
Attachments
(4 files, 2 obsolete files)
I noticed that the Devices list in Send Tab is showing the previous generation of icons. They should be using the Photon device icons like inside the app.
https://design.firefox.com/icons/viewer/#device
tracking-fxios:
--- → ?
Reporter | ||
Comment 1•7 years ago
|
||
If the tablet is not yet on the icon site, here it is.
status-firefox5:
? → ---
Send To in new share sheet has new icons, but Send To through the page menu has old ones. I am in this code ATM, will fix
Assignee: nobody → gkeeley
Flags: needinfo?(gkeeley)
Updated•6 years ago
|
tracking-fxios:
--- → ?
Can I get these as 24, 48, and 72 px high pngs? All iOS icons are PNGs at 1x, 2x, and 3x sizes.
The width must be the same, whatever common px width fits all the images.
Flags: needinfo?(gkeeley) → needinfo?(rfeeley)
Reporter | ||
Comment 4•6 years ago
|
||
I think these are the sizes you are looking for. 72 is too big.
Flags: needinfo?(rfeeley)
The current code expects 24, 48, and 72 px high PNGs. All the device images need to be correctly proportional, so if one image has 5 px of transparent border on top and bottom, another image can't be cropped content appears larger. Viewing the images side-by-side at the same zoom level is a way to check this.
Updated•6 years ago
|
Priority: -- → P2
Robin, can you locate/generate some icons for this? Thanks.
Flags: needinfo?(randersen)
Comment 8•6 years ago
|
||
Attachment #8975999 -
Flags: review?(gkeeley)
It is unclear if this bug is asking only to update the 2 device icons we use, or to ensure we show the mobile device icon type, phone vs tablet, which requires adding a 3rd icon type.
Comment 10•6 years ago
|
||
It looks like we just need to update the icons. We updated these for Photon in the main app bundle but never did for the share extension.
Assignee | ||
Comment 11•6 years ago
|
||
Taking the bug back :), I hadn't read your comment on the PR until now, I'll build with the icons you provided and see if the UIImageViews need adjustig
Assignee: fpatel → gkeeley
Assignee | ||
Comment 12•6 years ago
|
||
I am adding a build step to copy shared icons between projects.
The alternative is to have a SharedImages.xcassets (included in Client and ShareTo build targets), however this will break our current asset organization which is spread across Client Images.xcassets, Home xcassets, and Menu xcassets. I think moving individual images from each one of these into SharedImages.xcassest will be confusing to those looking for images in those asset groups.
Assignee | ||
Comment 13•6 years ago
|
||
Attachment #8975999 -
Attachment is obsolete: true
Attachment #8975999 -
Flags: review?(gkeeley)
Attachment #8976294 -
Flags: review?(fpatel)
Assignee | ||
Comment 14•6 years ago
|
||
We should show both types of mobile device icons, bug added here: https://bugzilla.mozilla.org/show_bug.cgi?id=1462113
Assignee | ||
Comment 15•6 years ago
|
||
This is a simpler PR that shares only the Send To Device assets, and this time use Xcode's build-in way of sharing assets.
Attachment #8976294 -
Attachment is obsolete: true
Attachment #8976294 -
Flags: review?(fpatel)
Attachment #8976551 -
Flags: review?(fpatel)
Updated•6 years ago
|
Attachment #8976551 -
Flags: review?(fpatel) → review+
Updated•6 years ago
|
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(randersen)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•