@atlaskit/react-ufo 2.4.2 → 2.4.4

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @atlaskit/ufo-interaction-ignore
 
+## 2.4.4
+
+### Patch Changes
+
+- [`a03da52505965`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/a03da52505965) -
+  Remove the flag from test files as well as its not present on LD or Statsig
+
+## 2.4.3
+
+### Patch Changes
+
+- [#171491](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/171491)
+  [`85cd3e428869a`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/85cd3e428869a) -
+  observe attributes VC90 impact via ufo:vc:next
+
 ## 2.4.2
 
 ### Patch Changes

package/dist/cjs/vc/vc-observer/observers/index.js

@@ -80,7 +80,7 @@ var Observers = exports.Observers = /*#__PURE__*/function () {
         renderStop: -1
       };
       (_this$mutationObserve = this.mutationObserver) === null || _this$mutationObserve === void 0 || _this$mutationObserve.observe(document.body, {
-        attributeFilter: ['hidden', 'style', 'src'],
+        attributeFilter: (0, _platformFeatureFlags.fg)('platform-ufo-vc-observe-attributes') ? ['hidden', 'style', 'src', 'class'] : ['hidden', 'style', 'src'],
         attributeOldValue: true,
         attributes: true,
         childList: true,
@@ -196,12 +196,8 @@ var Observers = exports.Observers = /*#__PURE__*/function () {
             });
           } else if (mutation.type === 'attributes') {
             if (mutation.target instanceof HTMLElement) {
-              // Commenting the following line temporarily.
-              // this.observeElement(mutation.target, mutation, 'attr', ignoreReason);
-
-              // using the if below to collect data
-              // how would VC90 changes if only style display changes are accounted for
-              if (mutation.attributeName === 'style' && _this2.getStyleDisplay(mutation.target.getAttribute('style')) !== _this2.getStyleDisplay(mutation.oldValue)) {
+              var isDisplayStyleAttributeMutation = mutation.attributeName === 'style' && _this2.getStyleDisplay(mutation.target.getAttribute('style')) !== _this2.getStyleDisplay(mutation.oldValue);
+              if ((0, _platformFeatureFlags.fg)('platform-ufo-vc-observe-attributes') || isDisplayStyleAttributeMutation) {
                 _this2.observeElement(mutation.target, mutation, 'attr', ignoreReason);
               }
             }

package/dist/es2019/vc/vc-observer/observers/index.js

@@ -62,7 +62,7 @@ export class Observers {
       renderStop: -1
     };
     (_this$mutationObserve = this.mutationObserver) === null || _this$mutationObserve === void 0 ? void 0 : _this$mutationObserve.observe(document.body, {
-      attributeFilter: ['hidden', 'style', 'src'],
+      attributeFilter: fg('platform-ufo-vc-observe-attributes') ? ['hidden', 'style', 'src', 'class'] : ['hidden', 'style', 'src'],
       attributeOldValue: true,
       attributes: true,
       childList: true,
@@ -164,12 +164,8 @@ export class Observers {
           });
         } else if (mutation.type === 'attributes') {
           if (mutation.target instanceof HTMLElement) {
-            // Commenting the following line temporarily.
-            // this.observeElement(mutation.target, mutation, 'attr', ignoreReason);
-
-            // using the if below to collect data
-            // how would VC90 changes if only style display changes are accounted for
-            if (mutation.attributeName === 'style' && this.getStyleDisplay(mutation.target.getAttribute('style')) !== this.getStyleDisplay(mutation.oldValue)) {
+            const isDisplayStyleAttributeMutation = mutation.attributeName === 'style' && this.getStyleDisplay(mutation.target.getAttribute('style')) !== this.getStyleDisplay(mutation.oldValue);
+            if (fg('platform-ufo-vc-observe-attributes') || isDisplayStyleAttributeMutation) {
               this.observeElement(mutation.target, mutation, 'attr', ignoreReason);
             }
           }

package/dist/esm/vc/vc-observer/observers/index.js

@@ -73,7 +73,7 @@ export var Observers = /*#__PURE__*/function () {
         renderStop: -1
       };
       (_this$mutationObserve = this.mutationObserver) === null || _this$mutationObserve === void 0 || _this$mutationObserve.observe(document.body, {
-        attributeFilter: ['hidden', 'style', 'src'],
+        attributeFilter: fg('platform-ufo-vc-observe-attributes') ? ['hidden', 'style', 'src', 'class'] : ['hidden', 'style', 'src'],
         attributeOldValue: true,
         attributes: true,
         childList: true,
@@ -189,12 +189,8 @@ export var Observers = /*#__PURE__*/function () {
             });
           } else if (mutation.type === 'attributes') {
             if (mutation.target instanceof HTMLElement) {
-              // Commenting the following line temporarily.
-              // this.observeElement(mutation.target, mutation, 'attr', ignoreReason);
-
-              // using the if below to collect data
-              // how would VC90 changes if only style display changes are accounted for
-              if (mutation.attributeName === 'style' && _this2.getStyleDisplay(mutation.target.getAttribute('style')) !== _this2.getStyleDisplay(mutation.oldValue)) {
+              var isDisplayStyleAttributeMutation = mutation.attributeName === 'style' && _this2.getStyleDisplay(mutation.target.getAttribute('style')) !== _this2.getStyleDisplay(mutation.oldValue);
+              if (fg('platform-ufo-vc-observe-attributes') || isDisplayStyleAttributeMutation) {
                 _this2.observeElement(mutation.target, mutation, 'attr', ignoreReason);
               }
             }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/react-ufo",
-	"version": "2.4.2",
+	"version": "2.4.4",
 	"description": "Parts of React UFO that are publicly available",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -130,6 +130,9 @@
 		"platform-ufo-invisible-element-vc-calculations": {
 			"type": "boolean"
 		},
+		"platform-ufo-vc-observe-attributes": {
+			"type": "boolean"
+		},
 		"react-ufo-payload-segment-compressed": {
 			"type": "boolean"
 		},