meteor-editable-text icon indicating copy to clipboard operation
meteor-editable-text copied to clipboard
verified publisher icon JackAdams

eventType not working

Open txs opened this issue 10 years ago • 7 comments

I followed the doc and mousein doesn't work and the mouseover or mouseenter don't work either! Hope to fix this!

txs avatar Sep 23 '15 11:09 txs

Thanks for reporting. There's definitely something going wrong -- I'll look into it.

JackAdams avatar Sep 23 '15 23:09 JackAdams

Unfortunately, this needs a pretty substantial refactor to get it going properly. For the meantime, it's not going to support any events except "click", "dblclick", "mousedown". I've changed the docs to reflect this.

JackAdams avatar Sep 23 '15 23:09 JackAdams

It's ok I use the meteor events to sent the fake one. However, you should change the doc to tell people not to use trigger. it triggers too many times. However, without trigger seams fine.

2015-09-24 7:48 GMT+08:00 Brent Abrahams [email protected]:

Unfortunately, this needs a pretty substantial refactor to get it going properly. For the meantime, it's not going to support any events except "click", "dblclick", "mousedown". I've changed the doc to reflect this.

— Reply to this email directly or view it on GitHub https://github.com/JackAdams/meteor-editable-text/issues/52#issuecomment-142762438 .

txs avatar Sep 24 '15 18:09 txs

Yes, you're right -- the editable-text-trigger does cause recursion for any other event types.

JackAdams avatar Sep 24 '15 22:09 JackAdams

please change the readme

2015-09-25 6:59 GMT+08:00 Brent Abrahams [email protected]:

Yes, you're right -- the editable-text-trigger does cause recursion for any other event types.

— Reply to this email directly or view it on GitHub https://github.com/JackAdams/meteor-editable-text/issues/52#issuecomment-143071696 .

txs avatar Sep 27 '15 19:09 txs

Added a warning to the README. Leaving this issue open as those events like mouseover, mouseenter should really be supported as well. It's just not going to happen anytime in the near future, unfortunately.

JackAdams avatar Oct 05 '15 13:10 JackAdams

No problem. I did a hack myself :D

2015-10-05 21:59 GMT+08:00 Brent Abrahams [email protected]:

Added a warning to the README. Leaving this issue open as those events like mouseover, mouseenter should really be supported as well. It's just not going to happen anytime in the near future, unfortunately.

— Reply to this email directly or view it on GitHub https://github.com/JackAdams/meteor-editable-text/issues/52#issuecomment-145535918 .

txs avatar Oct 14 '15 08:10 txs