Skip to content
This repository was archived by the owner on Feb 26, 2024. It is now read-only.

fix(addEventListener): when called from the global scope#191

Merged
vicb merged 3 commits intoangular:masterfrom
vicb:1005-evl
Oct 6, 2015
Merged

fix(addEventListener): when called from the global scope#191
vicb merged 3 commits intoangular:masterfrom
vicb:1005-evl

Conversation

@vicb
Copy link
Copy Markdown
Contributor

@vicb vicb commented Oct 6, 2015

fixes #190

@IgorMinar / @btford could you please review.

I'll check within the context of Angular & report here

/ref angular/angular#4531

@vicb
Copy link
Copy Markdown
Contributor Author

vicb commented Oct 6, 2015

Also see the discussion in #184

/cc @matthewjh

@vicb vicb force-pushed the 1005-evl branch 2 times, most recently from 7f5079d to 24941bf Compare October 6, 2015 06:10
@vicb
Copy link
Copy Markdown
Contributor Author

vicb commented Oct 6, 2015

Status:

  • There is one issue with this PR on Android,
  • I can get my Angular env to work (most probably related to the node/npm upgrades).

I'll try to solve those tomorrow.

Comment thread dist/zone-microtask.js Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove the reference to "self", it only confuses the reader

@IgorMinar
Copy link
Copy Markdown
Contributor

otherwise lgtm

@vicb vicb merged commit 22dce10 into angular:master Oct 6, 2015
@vicb vicb deleted the 1005-evl branch October 6, 2015 17:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

addEventListener in strict mode

2 participants