Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

eventType not working #52

Open
txs opened this issue Sep 23, 2015 · 7 comments
Open

eventType not working #52

txs opened this issue Sep 23, 2015 · 7 comments

Comments

@txs
Copy link

txs commented Sep 23, 2015

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

@JackAdams
Copy link
Owner

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

@JackAdams
Copy link
Owner

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.

@txs
Copy link
Author

txs commented Sep 24, 2015

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
#52 (comment)
.

@JackAdams
Copy link
Owner

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

@txs
Copy link
Author

txs commented Sep 27, 2015

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
#52 (comment)
.

@JackAdams
Copy link
Owner

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.

@txs
Copy link
Author

txs commented Oct 14, 2015

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
#52 (comment)
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants