Skip to content

Conversation

sis0k0
Copy link
Contributor

@sis0k0 sis0k0 commented Jul 2, 2018

Should be merged with rebase and merge to keep the commit messages.
Changes:

  • fix: provide NullViewportScroller in NativeScriptModule
    Required by: angular/angular@49c5234

  • fix: remove global document object
    BREAKING CHANGES:
    The document object is no longer property of the global object. This
    may cause behavioral changes in some plugin that use the document
    object to determine if they're running in browser context.
    fixes Global 'document' polyfill causes crashes when used with some node packages #1144

  • refactor: remove INTERNAL_BROWSER_PROVIDERS from the compiler providers
    They rely on the globally attached document object. Also, they're no longer required.

related to #1398

@ghost ghost assigned sis0k0 Jul 2, 2018
@ghost ghost added the in progress label Jul 2, 2018
@sis0k0 sis0k0 added this to the 6.1 (TBD) milestone Jul 2, 2018
sis0k0 added 5 commits July 2, 2018 13:14
BREAKING CHANGES:

The `document` object is no longer property of the `global` object. This
may cause behavioral changes in some plugin that use the `document`
object to determine if they're running in browser context.

fixes #1144
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.

Global 'document' polyfill causes crashes when used with some node packages
2 participants