Closed Bug 946268 Opened 11 years ago Closed 10 years ago

[Music] Artifacts below music scrubber

Categories

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

x86
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: epang, Assigned: pivanov)

References

Details

(Whiteboard: ux-tracking, visual design, visual-tracking, bokken)

Attachments

(1 file, 3 obsolete files)

Attached image music player.png (obsolete) —
Hey Pavel,

The right side of the scrubber highlight is a bit cut off.  This wasn't occurring before, was it a regression from the fix we did for WVGA? Can you take a look?

Thanks!
Attached file patch for Gaia/master (obsolete) —
Hey Eric, 
there are some glitches because of odd sizes ... but I think that I fix it for now, can you check it ?
Attachment #8342553 - Flags: review?(epang)
(In reply to Pavel Ivanov [:ivanovpavel] from comment #1)
> Created attachment 8342553 [details] [review]
> patch for Gaia/master
> 
> Hey Eric, 
> there are some glitches because of odd sizes ... but I think that I fix it
> for now, can you check it ?

glitches still seem to be happening, I'll attach a screen that i took from wvga.  You can see the right side of the srubber looks cut off.  The glitches seem to change when you move the slider around.
Flags: needinfo?(pivanov)
Attached image music player.png (obsolete) —
example image of the glitches in the wvga music player.  Can you take a look?  We need to land this by tomorrow.
this is because the odd sizes :( I think we don't have time to fix this because is deeper than just CSS
Flags: needinfo?(pivanov)
(In reply to Pavel Ivanov [:ivanovpavel] from comment #4)
> this is because the odd sizes :( I think we don't have time to fix this
> because is deeper than just CSS

is there anything I can do about the odd sizes?  I can always try to update the designs to help. Let me know, thanks!
Flags: needinfo?(pivanov)
Can we try with something like 2.2 instead of 2.1 and 6.2 instead of 6.1?
Flags: needinfo?(pivanov)
(In reply to Pavel Ivanov [:ivanovpavel] from comment #6)
> Can we try with something like 2.2 instead of 2.1 and 6.2 instead of 6.1?

sure,let's give it a try. thanks!
Flags: needinfo?(pivanov)
Hey Eric can you test the patch now ... I think that I found a way to hack it and I use 2.3rem and 6.1rem like in spec
Flags: needinfo?(pivanov)
Flags: needinfo?(epang)
Attached image Music Player.png
Hey Pavel, the scrubber looks good now! But not when you move it around lines appear, see the attached image.  Do you know what's happening here?  This only occurs in WVGA 1.5x.
Attachment #8342424 - Attachment is obsolete: true
Attachment #8342876 - Attachment is obsolete: true
Flags: needinfo?(epang) → needinfo?(pivanov)
huh ... no idea ... maybe we need to wait till we have devices to test
Flags: needinfo?(pivanov)
Summary: [Music] Right edge of music scrubber highlight is cut off → [Music] Artifacts below music scrubber
Blocks: SysFE
No longer blocks: 1.3-visual-refresh
Whiteboard: ux-tracking, visual design, visual-tracking, jian → ux-tracking, visual design, visual-tracking, bokken
Attachment #8342553 - Flags: review?(epang)
Hey Eric,
can you test the PR now ... I can't reproduce the problem on a peak (HD device) a have a chance to test it on it
Flags: needinfo?(epang)
(In reply to Pavel Ivanov [:ivanovpavel] from comment #11)
> Hey Eric,
> can you test the PR now ... I can't reproduce the problem on a peak (HD
> device) a have a chance to test it on it

Hey Pavel, I just tested on the helix and I'm unable to reproduce the problem too.  Did you fix?
Flags: needinfo?(epang)
Comment on attachment 8342553 [details] [review]
patch for Gaia/master

Nope,

I think there is a platform fix for the graphics and that's why now looks fine :)
Attachment #8342553 - Attachment is obsolete: true
(In reply to Pavel Ivanov [:ivanovpavel] from comment #13)
> Comment on attachment 8342553 [details] [review]
> patch for Gaia/master
> 
> Nope,
> 
> I think there is a platform fix for the graphics and that's why now looks
> fine :)

ok, we should land this :). Thanks!
Work fine now
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: