bem / bem-bl

Base BEM library
http://bem.github.com/bem-bl/
198 stars 84 forks source link

BEM-2242 – Disable fastclick for link on ios7,8. #638

Closed Yeti-or closed 9 years ago

Yeti-or commented 9 years ago

@xoxulin review plz

xoxulin commented 9 years ago

:ok:

arikon commented 9 years ago

@Yeti-or Это не почините в свежих версиях fastclick? Может, либу обновить?

Yeti-or commented 9 years ago

@arikon нет в свежих fastclick фикса нет. Но я бы завел задачу на портирование полезностей из апстрима в i-fastclick

arikon commented 9 years ago

@Yeti-or Заведи, пожалуйста. Это нужно делать время от времени =)

narqo commented 9 years ago

А вы в fastclick задачу про это завели? Рано или поздно получится, что у вас форк, в котором в шапке файла написано, что это fastclick@1.0.3, но никто не помнит, что именно тут от оригинальной версии, а что нет. Со ссылками на воркэраунды к конкретным задачам, можно будет хотябы следить за прогрессом, что еще нужно, а что уже есть в апстриме

Yeti-or commented 9 years ago

Воспроизводится в fastclick#1.0.6 завел: https://github.com/ftlabs/fastclick/issues/391

Yeti-or commented 9 years ago

Решил что нужно отключать только с targer='_blank' https://github.com/bem/bem-bl/pull/640