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.

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

Add @-ms-viewport for IE10 #85

Merged
merged 1 commit into from
Nov 21, 2013
Merged

Conversation

nschonni
Copy link
Member

Test cases taken from the documentation in
http://msdn.microsoft.com/en-us/library/ie/hh869615%28v=vs.85%29.aspx
Fixes CSSLint/csslint#401


This currently fails for the case of a viewport nested in a media query http://dev.w3.org/csswg/css-device-adapt/#media-queries
I'm throwing this up so maybe someone (ping @jklein) can figure out a good solution.

@nschonni
Copy link
Member Author

nschonni commented Sep 4, 2013

@nzakas any thoughts on this? Should I just remove the offending test and refile that as a separate issue? The failure seems more related to the @viewport code, rather than the additional ms version

@nzakas
Copy link
Contributor

nzakas commented Sep 7, 2013

Yeah, adding a failing test without a fix means I can't merge it in.

@nschonni
Copy link
Member Author

nschonni commented Sep 7, 2013

I removed the failing test, and will file it as a separate issue

@nschonni
Copy link
Member Author

@nzakas is there anything that needs to be updated for this to land?

@nzakas
Copy link
Contributor

nzakas commented Nov 21, 2013

Sorry, just lost track of it.

nzakas added a commit that referenced this pull request Nov 21, 2013
@nzakas nzakas merged commit f9f34c1 into CSSLint:master Nov 21, 2013
@nschonni nschonni deleted the add-ms-viewport branch November 22, 2013 03:59
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

Successfully merging this pull request may close these issues.

@-ms-viewport - is an error
2 participants