feat(iron-autogrow-textarea.html): add debounce function #117
Conversation
|
I don't think this is good, since it basically delays everything by half a second. In particular, this also breaks all the tests, which means it breaks the tests of anybody who's ever written a test for using this element in their app. |
| this.$.mirror.innerHTML = this._valueForMirror(); | ||
| this.debounce('delayMirror', function() { | ||
| this.$.mirror.innerHTML = this._valueForMirror(); | ||
| }.bind(this), 500); |
There was a problem hiding this comment.
can we use no wait at all?
There was a problem hiding this comment.
I can try without the wait, to see if it works.
There was a problem hiding this comment.
It does not work without the wait. I've obtain similar results that without the debounce function.
I've also tested different values of waiting, but with 500 I've obtain better results and the delay on typing desappear:
My scenery is like a chat (made with components): a user enters in a room from a list on the left, and starts typing messages. The most number of rooms you have on the left, the greater delay you feel.
In my case, the bottleneck is this line:
this.$.mirror.innerHTML = this._valueForMirror();
Maybe this line can be made in a different way and so the debounce would not be necessary.
This fixes #116, adding a
debouncefunction to avoid the delay to the user.