this overwritten as component name in arguments of hoisted event handler

[Conduitry]

See 2.1.1 vs 2.2.0.

This might be as simple as wrapping this in a check for !this.shouldHoist, but not sure whether that's the best solution. Maybe we should instead make sure this.shouldHoist is false if this is one of the arguments to the handler?

cc @stalkerg

[Conduitry]

I realised the other day that the 'is this a descendant of an each block or not?' test doesn't really make sense, since the component itself could be in an each block

@Rich-Harris Sooo ... what does this mean for what we want to do here? Are you suggesting that we always want to hoist event handlers (unless it's a custom element)?

[Rich-Harris]

I think we should prefer hoisting, yeah. I think we probably only want to overwrite this if it's a custom event

[stalkerg]

@Conduitry thanks for this issue, looks like my gitter doesn't work.

[Rich-Harris]

@stalkerg mine too, I think it's broken 😱

[Rich-Harris]

fixed in 2.4.1