Brent Abrahams

Results 123 comments of Brent Abrahams

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...

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

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...

Good point. I won't change parseInt to parseFloat.

Yes, that's right. It's not something I've ever needed, so I haven't implemented it. The code base is a bit messy but if you want to do a PR I'll...

Yes, that would work, but use `beforeUpdate` rather than `beforeInsert`. (The widget only inserts a whole document if it has `autoInsert=true` set and a context that doesn't contain an `_id`...

Yeah, it's undocumented because it's not very well implemented. There are a few other undocumented bits and pieces too.

I'll accept a PR if you get a workable solution going. The relevant lines of code are: https://github.com/JackAdams/meteor-editable-text/blob/master/lib/editable_text.js#L394-L403

In meteorpad.com, no events are triggered on the hover event -- that's just a cleverly styled `input` tag with some css-hover effects. But, yes, I've thought a lot about making...

Wrapping the editable text in anchor tags could certainly lead to some unwelcome side effects. I took a quick look at your repo for crowducate: ``` {{# if editingThisCourse }}...