Skip to content

border-radius is not prefixed #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
mkantor opened this issue Feb 10, 2012 · 8 comments
Closed

border-radius is not prefixed #12

mkantor opened this issue Feb 10, 2012 · 8 comments

Comments

@mkantor
Copy link
Contributor

mkantor commented Feb 10, 2012

The border-radius shorthand property doesn't seem to be getting prefixed correctly. However, border-top-left-radius, border-top-right-radius, etc work fine.

@codler
Copy link
Owner

codler commented Feb 10, 2012

What browser are you testing? Most of the browser doesn't need prefixing border-radius anymore. http://caniuse.com/#feat=border-radius

@mkantor
Copy link
Contributor Author

mkantor commented Feb 13, 2012

I noticed it in Firefox 3.6, is that browser not supported?

Sorry if this is getting off-topic, but do you have a list of supported/tested browsers?

@mkantor
Copy link
Contributor Author

mkantor commented Feb 13, 2012

Also occurs in Safari 4 (as indicated by http://caniuse.com/#feat=border-radius ).

@codler
Copy link
Owner

codler commented Feb 13, 2012

I usually only test on latest stable version of browsers ( and IE8 ) since they have auto update and will only support those.

Version 1.39 was the last time I tested on FF 3.6 but that version only works up to jQuery 1.5.
Have you an example of how your border-radius shorthand looks like?
Could it be because of this?
See under <percentage> values section https://developer.mozilla.org/en/css/-moz-border-radius#section_8

@mkantor
Copy link
Contributor Author

mkantor commented Feb 14, 2012

I'm using px.

Here's a test case:
http://jsfiddle.net/afEwj/1/

And here's a screenshot of the test case in both FF 3.6 and Safari 4:
http://i.imgur.com/l17Iy.png

Anyway if these browsers aren't supported that's understandable, but it'd be handy if the README included all version numbers in the list of tested browsers.

@codler
Copy link
Owner

codler commented Feb 14, 2012

aha, border-radius: 10px; have worked before in FF 3.6.

Could you check if this works?
http://jsfiddle.net/afEwj/2/

@mkantor
Copy link
Contributor Author

mkantor commented Feb 14, 2012

Success!
http://i.imgur.com/ICJPr.png

@codler
Copy link
Owner

codler commented Feb 14, 2012

Thanks!

@codler codler closed this as completed Feb 14, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants