make elastic optional again about jquery-mentions-input HOT 7 CLOSED

just remove the code "elmInputBox.elastic();" from the js. it should work that way !

but don't forget, you will face a problems with the the highlight. so it's better to stay using elastic or to remove the highlighting :D

from jquery-mentions-input.

Sure, but the idea is to make as few modifications to upstream code as possible, in order to make a simple update of a js lib not a possible source for regressions.

from jquery-mentions-input.

+1

I'd like to know the reasoning for putting elastic in this. It seems unneeded. But if I remove that elastic() line from the source it doesn't work anymore.

from jquery-mentions-input.

The reason for why the mentionsInput needs to be "elastic", is because the highlighting of is happening in an underlay underneath the textarea.

If the textarea is having a fixed height, at some point scrollbars will appear, and then the position of the underlay would need to be update. This would require some syncing of positions triggered by the scroll-event, and I imagine it would be a mess, and have bad performance.

So currently if we remove, or disable the elastic-ness, content in the textarea and the underlay would be out of sync when scrolling.

from jquery-mentions-input.

+1

from jquery-mentions-input.

How about making it an undocumented, non-default option?
So that those who want to use a different auto-resize plugin can do so, without having to modify this plugin (even if it's just one line that needs to be commented out).
;)

from jquery-mentions-input.

I re-added the option. a60ab5c

from jquery-mentions-input.