800002 - [Gallery] Edit Mode - Exposure Polish

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Gallery, defect, P3)

Description

[Patryk Adamczyk [:patryk] UX]

Can you please make the slide match the visual design: https://www.dropbox.com/s/771lbuva8lb2hij/Gallery_Edit_Exposure_A_120720.jpg

Comment 1

[David Flanagan [:djf]]

I cannot match that kind of thing by eye. It looks to me like you've got a gray gradient, but I can't tell what it is. And I have no idea how I'd do the 'groove' of the slider with CSS.  If you give me an image to use as the slider background, I can use that. Or if you give me precise specs on the background color, I can use those.  But without details I'm just going to screw around and get something that looks okay to me and probably looks terrible to you. 

Also, the draggable part of the slider is too small to reliably grab as you've designed it, so I'll have to make the touch-sensitive area of it bigger than the visible area.

Comment 2

[Patryk Adamczyk [:patryk] UX]

Attachment: Crop Spec

Comment 3

[Patryk Adamczyk [:patryk] UX]

Added a spec for the new crop tool. Cut up all the graphics here:
https://www.dropbox.com/sh/grmnjr3eqexfkur/O3wIoOPD4X

Hope this helps!

Comment 4

[Eric Pang [:epang] UX]

David, just checked the latest gaia/master build (12-03-2013).  It doesn't look like this has been implemented yet.  What's the status on this? Let me know if you need any more details or specs, thanks!

Comment 5

[David Flanagan [:djf]]

I don't think I need anything more except more time to work on stuff like this. This would be a good bug for someone else to take...

Comment 6

[Eric Pang [:epang] UX]

Hi Sam, have you started work on this bug? Let me know, thanks!

Comment 7

[Sam Joch [:samjoch]]

Attachment: PR Github

Comment 8

[Sam Joch [:samjoch]]

Attachment: After patch screenshot

Comment 9

[Eric Pang [:epang] UX]

Comment on attachment 760437 [details]
After patch screenshot

This looks good. Thanks Sam!

Comment 10

[Dale Harvey (:daleharvey)]

Comment on attachment 760436 [details]
PR Github

As its gallery, passing review along to David

Comment 11

[David Flanagan [:djf]]

Comment on attachment 760436 [details]
PR Github

There are some nits on github that I'd like you to address, but overall the patch seems to work fine, and I'm giving r+.

It seems to me that the slider line and the vertical tick marks lines are too faint to see well in bright light, but that's a visual design issue.

Also: by changing the background color from gray to black, you basically break the two black border options, so please file a followup bug to remove those border options.  Perhaps we should just have four options for four different white border widths? We'll need to consult with visual design, I suppose.

Comment 12

[Eric Pang [:epang] UX]

Hey Sam, what happened with this?  Are we able to land this?

Comment 13

[Sam Joch [:samjoch]]

Landed in master: https://github.com/mozilla-b2g/gaia/commit/474e067e2ad26d5ce97a052a681fee92273e3a16