akre54 / Backbone.NativeView

A reference implementation of a native Backbone.View
MIT License
113 stars 18 forks source link

2 Fixes #28

Closed cloakedninjas closed 7 years ago

cloakedninjas commented 7 years ago

If you're interested, here's 2 commits to fix:

26 and #23

paulfalgout commented 4 years ago

Not delegating the focus/blur events just means that re-rendering the view will leave orphaned listeners and the newly rendered view won't have the events. This should have used the useCapture flag for these events

akre54 commented 4 years ago

@paulfalgout got a fix?

paulfalgout commented 4 years ago

@akre54 playing with it now. Just realized the issue

akre54 commented 4 years ago

Want to open a pull?

paulfalgout commented 4 years ago

😃 . Yeah I put it on my list. I'm not in a hurry though.

akre54 commented 4 years ago

Okay, happy to take a look whenever you have something ready