psergus / ngWYSIWYG

true angular WYSIWYG
MIT License
63 stars 42 forks source link

Incremental click event listeners amount #78

Open nikaivan opened 7 years ago

nikaivan commented 7 years ago

Hello,

We're implementing ngwysiwyg on an AngularJS project that uses angular-material.

Angular material has a feature that allow us to split the content on different tabs. Related to this, we found a problem when using ngwysiyg inside a tab.

In file wysiwygEdit.js line 21, is defined an onclick event listener, that fires "NGP_EVENTS.CLICK_AWAY" function, whose functionality is to hide symbols dialog when the users click outside the dialog.

That event listener is created every time the user changes from one tab to another, resulting in an incremental number of event listeners, and in a huge lag, and in some cases in the freeze of the internet browser.

Temporary we've commented NGP_EVENTS.CLICK_AWAY listener creation, for fixing it!

Regards.