Skip to content
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.

Sign up for GitHub

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

Jump to bottom

vendor prefixes! #305

Closed
gollodev opened this issue Jun 10, 2015 · 5 comments · Fixed by #306
Closed

vendor prefixes! #305

gollodev opened this issue Jun 10, 2015 · 5 comments · Fixed by #306

Comments

@gollodev
Copy link

add this line vendor-prefixes ?= webkit moz o ms official, only shows webkit...
@notslang
Copy link
Collaborator

I'm going to need a bit more info to reproduce that... maybe a code sample?

@gollodev
Copy link
Author 

button
    border-radius 2px

compiled...

button {
  -webkit-border-radius: 2px;
  border-radius: 2px;
}

should be add -moz- -o- prefixes no?

@notslang
Copy link
Collaborator

As far as I can tell, opera never had a prefix for border-radius, and FF hasn't needed one since 3.6. Actually, Chrome hasn't needed one since v4, so we probably shouldn’t even be adding the webkit prefix.

@netlemur
Copy link
Contributor

Yes, the webkit prefix should be removed for border-radius.

@netlemur netlemur mentioned this issue Jun 11, 2015
Merged
@netlemur
Copy link
Contributor

@slang800 in case that's something you guys wanna do, there's a PR for it now: #306

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove vendor prefix for border-radius
3 participants
@netlemur @notslang @gollodev