Closed
Bug 1023117
Opened 10 years ago
Closed 10 years ago
[Flame][Camera]Missing images for Camera
Categories
(Firefox OS Graveyard :: Gaia::Camera, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: HHuang, Assigned: pivanov)
References
Details
Attachments
(3 files)
(deleted),
application/zip
|
Details | |
(deleted),
text/x-github-pull-request
|
wilsonpage
:
review+
HHuang
:
ui-review+
|
Details |
(deleted),
image/png
|
Details |
Need to update @2.25x missing images for Camera.
Reporter | ||
Comment 1•10 years ago
|
||
Hi Pavel, I've updated following assets, please upload them, thanks! ./apps/camera/style/images/video_pause_button@2.25x.png ./apps/camera/style/images/video_play_button@2.25x.png ./apps/camera/style/images/video_play_focus@2.25x.png ./apps/camera/style/images/video_play_normal@2.25x.png
Flags: needinfo?(pivanov)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → pivanov
Flags: needinfo?(pivanov)
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8437524 -
Flags: ui-review?(hhuang)
Reporter | ||
Comment 3•10 years ago
|
||
Comment on attachment 8437524 [details]
patch for Gaia/master
Thanks for helping!
Attachment #8437524 -
Flags: ui-review?(hhuang) → ui-review+
Assignee | ||
Updated•10 years ago
|
Attachment #8437524 -
Flags: review?(wilsonpage)
Comment 4•10 years ago
|
||
Comment on attachment 8437524 [details] patch for Gaia/master Looks good, except the pause icon on the video controls is missing. I'm building with: `$ make install-gaia APP=camera GAIA_DEV_PIXELS_PER_PX=2.25` Is that correct? I also noticed that `style/images/Video_pause_button@2x.png` looks like it has too much white-space. Could you check this out, perhaps fix with this issue?
Attachment #8437524 -
Flags: review?(wilsonpage)
Assignee | ||
Comment 5•10 years ago
|
||
Hey Wilson, thanks for the feedback ... you are absolutely right ... I think that I fixed the pause icon ... but we will need new version of `style/images/Video_pause_button@2x.png` because of white space furthermore the icon have shadow. Helen, can you help?
Flags: needinfo?(hhuang)
Reporter | ||
Comment 7•10 years ago
|
||
Hi Pavel, can you replace the image I attached?
Flags: needinfo?(pivanov)
Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8437524 [details]
patch for Gaia/master
think that we are OK now :) (the images are compressed too)
Attachment #8437524 -
Flags: review?(wilsonpage)
Updated•10 years ago
|
Attachment #8437524 -
Flags: review?(wilsonpage) → review+
Assignee | ||
Comment 10•10 years ago
|
||
Thanks :) Landed to master: https://github.com/mozilla-b2g/gaia/commit/c6bfe5ded873e4ea0d46654d2c005f643eae0d8a
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•