bem-bl icon indicating copy to clipboard operation
bem-bl copied to clipboard

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

Open narqo opened this issue 10 years ago • 4 comments

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 avatar May 14 '14 14:05 narqo

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

tadatuta avatar May 14 '14 14:05 tadatuta

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 :)

narqo avatar May 14 '14 15:05 narqo

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

tadatuta avatar May 14 '14 15:05 tadatuta

@tadatuta @narqo welcome to BEM-2125 :)

deeonis avatar Feb 25 '15 16:02 deeonis