Skip to content

Apply context when dispatching event#6

Merged
developit merged 1 commit intodevelopit:masterfrom
jamesbirtles:patch-1
Feb 20, 2017
Merged

Apply context when dispatching event#6
developit merged 1 commit intodevelopit:masterfrom
jamesbirtles:patch-1

Conversation

@jamesbirtles
Copy link
Contributor

@jamesbirtles jamesbirtles commented Feb 20, 2017

this inside of event listeners should be the element that the listener was registered on.

Without this, dispatching events on preact elements was failing with Cannot read property 'click' of undefined for example

`this` inside of event listeners should be the element that the listener was registered on.

Without this, dispatching events on preact elements was failing with `Cannot read property 'click' of undefined`
@developit
Copy link
Owner

[funny: I had typed out a whole reply about preact before I realized this was undom haha]

Happy to merge this, was laziness on my part to forget the first time around.

@developit developit merged commit 78addb9 into developit:master Feb 20, 2017
@jamesbirtles jamesbirtles deleted the patch-1 branch March 6, 2017 18:58
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

Successfully merging this pull request may close these issues.

2 participants