button: add styles for ie8 #1274
Conversation
sipayRT
force-pushed
the
issues/1205@v2_button
branch
from
8e2f59c
to
82bd1ce
Compare
sipayRT
force-pushed
the
issues/1205@v2_button
branch
from
0b1cb8d
to
880b07e
Compare
|
@mishanga Can you review this PR? |
| { elem : 'css', url : '_simple.css' }, | ||
| '<!--<![endif]-->', | ||
| { elem : 'css', url : '_simple.ie8.css', ie : 'lt IE 9' }, |
There was a problem hiding this comment.
А тут не должно быть просто 'IE 8'?
There was a problem hiding this comment.
Делалось только для "вдруг и в ie7 заработает". Хотя, если мы жестко укажем, что поддерживает только ie8, то можно и правда подключать только для ie8
There was a problem hiding this comment.
Ну тогда не ie8.css, а ie.css.
|
@mishanga попробовал все переделать без бордеров (сделал отдельным коммитом). Есть проблема с перерисовкой псевдоэлементов в ИЕ - при добавлении класса он не применяет изменение стилей для псевдоэлементов, приходится вручную вызывать reflow. При каждом изменении состояния нужно добавлять что-нибудь типа т.е. изменять содержимое пробелом. Получается какая-то каша из этих контентов |
|
Точно нельзя никак элегантнее сделать? |
|
Я пока не придумал. Без этого хака изменения применяются только при смене фокуса кнопки. |
|
@mishanga а при разработке "бабушки" вы каким образом кнопку делали? |
|
Там всё примитивнее: https://github.yandex-team.ru/serp/granny/blob/dev/blocks/button/button.css |
|
Тогда я все же склоняюсь к бордерам - меньше всяких хаков |
|
Ну ок. |
sipayRT
force-pushed
the
issues/1205@v2_button
branch
from
1c4d43a
to
48aa880
Compare
No description provided.