Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

Conversation

mkowalchuk
Copy link
Contributor

Fixes #8905

@crisbeto
Copy link
Member

LGTM @ThomasBurleson.

@crisbeto crisbeto added the pr: merge ready This PR is ready for a caretaker to review label Jun 30, 2016
@clshortfuse
Copy link
Contributor

Functionality-wise, it looks good. My only concern is the inconsistency of the variable name with other elements. If I'm not mistaken, mdAutocomplete has something similar, but there it's called ‘wait‘.

@crisbeto
Copy link
Member

crisbeto commented Jul 1, 2016

The option in autocomplete is called delay, but I think it's different. The one in autocomplete seems to be the delay before results start showing up, the changes in this PR are for the debounce period before the model is updated.

@clshortfuse
Copy link
Contributor

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants