Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use "window" instead of "Type" for the "global" substitutions #909

Closed
wants to merge 3 commits into from
Closed

Conversation

Martin-Luft
Copy link
Contributor

@Martin-Luft Martin-Luft commented Sep 1, 2016

use "window" instead of "Type" for the "global" substitutions

This was referenced Sep 1, 2016
@valorkin
Copy link
Member

valorkin commented Sep 1, 2016

Please, don't remove event hacks, it is for universal renderer

@codecov-io
Copy link

codecov-io commented Sep 1, 2016

Current coverage is 90.22% (diff: 100%)

Merging #909 into development will decrease coverage by 0.02%

@@           development       #909   diff @@
=============================================
  Files               25         25          
  Lines              769        767     -2   
  Methods            142        142          
  Messages             0          0          
  Branches           112        112          
=============================================
- Hits               694        692     -2   
  Misses              75         75          
  Partials             0          0          

Powered by Codecov. Last update 341dcf8...f00c5e3

@Martin-Luft
Copy link
Contributor Author

OK fixed :)

@valorkin
Copy link
Member

valorkin commented Sep 2, 2016

I have digged a bit dipper:
https://github.com/angular/angular/blob/ae75e3640a2d9eb1e897a0771d92b976c5a42c75/modules/@angular/facade/src/browser.ts

it is not so nice in in ng2 + angular universal will get working later,
may be we can drop this hack for now?

@valorkin
Copy link
Member

valorkin commented Sep 2, 2016

@Martin-Wegner fixed, tried to add your authorship but seems failed, sorry :)

@Martin-Luft Martin-Luft changed the title use "windows" instead of "Type" for the "global" substitutions use "window" instead of "Type" for the "global" substitutions Sep 2, 2016
@Martin-Luft
Copy link
Contributor Author

OK no problem :)

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.

3 participants