Merge pull request #2018 from NativeScript/release-8.2.2

Release 8.2.2

Author: elena-p

Diff for: CHANGELOG.md

@@ -1,3 +1,12 @@
+<a name="8.2.2"></a>
+## [8.2.2](https://github.com/NativeScript/nativescript-angular/compare/8.2.1...8.2.2) (2019-10-16)
+
+### Features
+
+* Support for adding scoped CSS without triggering global refresh ([#1999](https://github.com/NativeScript/nativescript-angular/pull/1999)) ([662c122](https://github.com/NativeScript/nativescript-angular/commit/662c122))
+
+
+
 <a name="8.2.1"></a>
 ## [8.2.1](https://github.com/NativeScript/nativescript-angular/compare/8.2.0...8.2.1) (2019-08-28)
 

Diff for: e2e/nested-router-tab-view/e2e/home-tabs.e2e-spec.ts

@@ -58,15 +58,20 @@ describe("home-tabs:", async function () {
                 await screen.navigateToTabsPage();
                 await screen.loadedTabs();
                 await screen.loadedPlayersList();
-                await screen.navigateToAboutPage();
-                await screen.loadedAbout();
-                await screen.loadedNestedAbout();
+
+                // TO DO: This is related to the tns-core-modules animations overhaul intiative (removing Animators in favor of androidx Transitions)
+                // Angular test case: navigating from Page with 3 nested frames and back crashes with "IllegalArgumentException: parameter must be a descendant of this view" only from these commented appium tests (does not reproduce with manual testing). This is fixed in androidx.fragment:fragment:1.2.0 and tests must be uncommented when migrating to it.
+                // await screen.navigateToAboutPage();
+                // await screen.loadedAbout();
+                // await screen.loadedNestedAbout();
             });
 
             it("should go back to Tabs and then back to Home", async function () {
-                await backActivatedRoute(driver);
-                await screen.loadedTabs();
-                await screen.loadedPlayersList();
+                // TO DO: This is related to the tns-core-modules animations overhaul intiative (removing Animators in favor of androidx Transitions)
+                // Angular test case: navigating from Page with 3 nested frames and back crashes with "IllegalArgumentException: parameter must be a descendant of this view" only from these commented appium tests (does not reproduce with manual testing). This is fixed in androidx.fragment:fragment:1.2.0 and tests must be uncommented when migrating to it.
+                // await backActivatedRoute(driver);
+                // await screen.loadedTabs();
+                // await screen.loadedPlayersList();
                 await backActivatedRoute(driver);
                 await screen.loadedHome();
                 await screen.loadedPlayersList();

Diff for: nativescript-angular/renderer.ts

@@ -356,7 +356,7 @@ class EmulatedRenderer extends NativeScriptRenderer {
     private addStyles(styles: (string | any[])[], componentId: string) {
         styles.map(s => s.toString())
             .map(s => replaceNgAttribute(s, componentId))
-            .forEach(addStyleToCss);
+            .forEach(addScopedStyleToCss);
     }
 }
 
@@ -365,6 +365,11 @@ const addStyleToCss = profile('"renderer".addStyleToCss', function addStyleToCss
     addCss(style);
 });
 
+// tslint:disable-next-line
+const addScopedStyleToCss = profile('"renderer".addScopedStyleToCss', function addScopedStyleToCss(style: string): void {
+    addCss(style, true);
+});
+
 function replaceNgAttribute(input: string, componentId: string): string {
     return input.replace(COMPONENT_REGEX, componentId);
 }