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

input: get rid of polling (close #287) #288

Open
wants to merge 1 commit into
base: v2
Choose a base branch
from
Open

input: get rid of polling (close #287) #288

wants to merge 1 commit into from

Conversation

dfilatov
Copy link
Member

@dfilatov dfilatov commented Feb 5, 2014

@narqo
Copy link
Member

narqo commented Feb 5, 2014

Надо проверить в Опере. У нас был недавно баг: если в форме воспользоваться кнопкой «Автозаполнение», Опера не триггерит вообще никаких событий :(

@dfilatov
Copy link
Member Author

dfilatov commented Feb 5, 2014

В какой опере? До хромиума?

@narqo
Copy link
Member

narqo commented Feb 5, 2014

В 12-й кнопка «логин» screen shot 2014-02-05 at 23 07 44 не триггерит ни input, ни change :(

В 19-й (вебкит) автозаполненная форма тоже ничего не триггерит (а еще и заполняется значениями до первого срабатывания поллинга :(

@dfilatov
Copy link
Member Author

dfilatov commented Feb 5, 2014

Какого поллинга?

@narqo
Copy link
Member

narqo commented Feb 5, 2014

Я имел ввиду, что тот код, который сейчас в v1 (где change триггерится через поллинг значения), тоже не отработает если браузер автозаполнит форму. Автозаполнение срабатывает по загрузке страницы (до первого события tick). Наверное на это надо отдельно смотреть/думать.

Другая проблема: этот тест теперь работает нечестно не только на тачах.

@mishanga
Copy link
Member

Здесь что-нибудь решили в итоге?

@dfilatov
Copy link
Member Author

Пока ничего определенного не решили. Надо решить что-то с автозаполнением.

@veged veged added the v2 label Jun 27, 2014
@deeonis deeonis added this to the v2.0 milestone Jul 24, 2014
@aristov
Copy link
Contributor

aristov commented Oct 13, 2014

resolves #287

@deeonis deeonis removed this from the v2.0 milestone Nov 12, 2014
@aristov
Copy link
Contributor

aristov commented Apr 2, 2015

Let's be honest and just close it with wontfix?

@narqo
Copy link
Member

narqo commented May 28, 2015

As discuss we need to move this poling logic into a separate modifier.

/cc @dfilatov

@km256
Copy link

km256 commented Jan 20, 2016

move this poling logic into a separate modifier.

+1

@qfox
Copy link
Member

qfox commented Jan 21, 2016

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants