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

b-page__css: Inconsistent behavior of ie-key for desktop and touch #512

Open
narqo opened this issue May 14, 2014 · 4 comments
Open

b-page__css: Inconsistent behavior of ie-key for desktop and touch #512

narqo opened this issue May 14, 2014 · 4 comments

Comments

@narqo
Copy link
Member

narqo commented May 14, 2014

For the same BEMJSON for b-page block we have different results

{ elem : 'css', url : '10-simple', ie : true }

For desktop:

<!--[if gt IE 9]><!--><link rel="stylesheet" href="_10-simple.css"/><!--<![endif]-->
<!--[if IE 6]><link rel="stylesheet" href="_10-simple.ie6.css"/><![endif]-->
<!--[if IE 7]><link rel="stylesheet" href="_10-simple.ie7.css"/><![endif]-->
<!--[if IE 8]><link rel="stylesheet" href="_10-simple.ie8.css"/><![endif]-->
<!--[if IE 9]><link rel="stylesheet" href="_10-simple.ie9.css"/><![endif]-->

For touch:

<!--[if gte IE 9]><link rel="stylesheet" href="_10-simple"/><![endif]-->

Note that we lost .css suffix on the touch level

@narqo narqo added the BEMHTML label May 14, 2014
@tadatuta
Copy link
Member

@narqo should it be fixed in major release or i can add something like indexOf('.css') > 0 as fallback and release patch version?

@narqo
Copy link
Member Author

narqo commented May 14, 2014

I don't mind if this would be fixed in minor release. But since nobody cares, one could wait for a major release without hacks :)

@tadatuta
Copy link
Member

@narqo If there's nobody who's affected let's postpone it till major release

@deeonis
Copy link
Contributor

deeonis commented Feb 25, 2015

@tadatuta @narqo welcome to BEM-2125 :)

@deeonis deeonis added the TODO label Feb 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants