volser / quill-table-ui

BSD 3-Clause "New" or "Revised" License
62 stars 15 forks source link

Menu is not hidden/opened properly #28

Open mnordlindh opened 1 year ago

mnordlindh commented 1 year ago

There is an issue with the menu not closing/updating its position due to an error in the docClickHandler not actually invoking the hideMenu function.

"docClickHandler = () => this.hideMenu;" should be
"docClickHandler = () => this.hideMenu();"

https://github.com/volser/quill-table-ui/blob/master/src/index.ts#L177

It works in version 1.0.5 (the demo) but not in version 1.0.7.

I can do a PR for this but as it is such a small change its probably easier for you to just update it. Let me know @volser.