489446 - Update videocontrols -moz-border-radius to match trunk

Status: VERIFIED FIXED

(Toolkit :: Video/Audio Controls, defect)

Description

[Justin Dolske [:Dolske]]

Attachment: Patch v.1

Due to the behavior noted in bug 474521, the -moz-border-radius used in the video control's CSS was 3px, instead of the 4px used on trunk. At the time this was needed to make 191 and trunk appear the same visually.

Whatever was causing the different rendering on 191 seems to have been fixed, and thus the relevant curved bits on 191 don't look correct anymore. This patch updates the CSS to patch the 4px value already used on trunk.

Comment 1

[Justin Dolske [:Dolske]]

Attachment: Screenshot / comparison

trunk on top, 1.9.1 (with this patch) on bottom. I also cut'n'pasted a bit of 191 over trunk, so that the changed areas are more readily comparable by just zooming in. As you can see, they're identical.

Comment 2

[Jennifer Morrow [:Boriss] (UX)]

Comment on attachment 373928 [details]
Screenshot / comparison

+ing

Comment 3

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 373925 [details] [diff] [review]
Patch v.1

a191=beltzner

Comment 4

[Justin Dolske [:Dolske]]

Pushed to 191: http://hg.mozilla.org/releases/mozilla-1.9.1/rev/ea774a371442

Comment 5

[Tony Chung [:tchung]]

Verified fix on Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US;
rv:1.9.2a1pre) Gecko/20090507 Minefield/3.6a1pre 
and Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b5pre)
Gecko/20090507 Shiretoko/3.5b5pre