Closed Bug 1023117 Opened 10 years ago Closed 10 years ago

[Flame][Camera]Missing images for Camera

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: HHuang, Assigned: pivanov)

References

Details

Attachments

(3 files)

Need to update @2.25x missing images for Camera.
Attached file Camera missing images.zip (deleted) —
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: nobody → pivanov
Flags: needinfo?(pivanov)
Attached file patch for Gaia/master (deleted) —
Attachment #8437524 - Flags: ui-review?(hhuang)
Blocks: 1016987
Comment on attachment 8437524 [details]
patch for Gaia/master

Thanks for helping!
Attachment #8437524 - Flags: ui-review?(hhuang) → ui-review+
Attachment #8437524 - Flags: review?(wilsonpage)
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)
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)
Attached image video_pause_button@2x.png (deleted) —
Sure! I've updated the image.
Flags: needinfo?(hhuang)
Hi Pavel, can you replace the image I attached?
Flags: needinfo?(pivanov)
yep ... done :)
Flags: needinfo?(pivanov)
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)
Attachment #8437524 - Flags: review?(wilsonpage) → review+
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.

Attachment

General

Created:
Updated:
Size: