Merge branch 'main' into fix/jump-links/missing-css-comma

patternfly · Sep 25, 2024 · 20c01eb · 20c01eb
2 parents 87b4f79 + 85f1854
commit 20c01eb
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 4 deletions.
diff --git a/core/pfe-core/CHANGELOG.md b/core/pfe-core/CHANGELOG.md
@@ -1,5 +1,11 @@
 # @patternfly/pfe-core
 
+## 4.0.2
+
+### Patch Changes
+
+- 0ec7338: `OverflowController`: prevent browser from locking up in some scenarios
+
 ## 4.0.1
 
 ### Patch Changes

diff --git a/core/pfe-core/controllers/overflow-controller.ts b/core/pfe-core/controllers/overflow-controller.ts
@@ -45,7 +45,9 @@ export class OverflowController implements ReactiveController {
   });
 
   #ro = new ResizeObserver(() => {
-    this.#setOverflowState();
+    requestAnimationFrame(() => {
+      this.#setOverflowState();
+    });
   });
 
   showScrollButtons = false;

diff --git a/core/pfe-core/package.json b/core/pfe-core/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/pfe-core",
-  "version": "4.0.1",
+  "version": "4.0.2",
   "license": "MIT",
   "description": "PatternFly Elements Core Library",
   "customElements": "custom-elements.json",

diff --git a/elements/pf-tabs/test/pf-tabs.spec.ts b/elements/pf-tabs/test/pf-tabs.spec.ts
@@ -199,11 +199,10 @@ describe('<pf-tabs>', function() {
       });
 
       beforeEach(nextFrame);
-      beforeEach(updateComplete);
+      beforeEach(nextFrame);
       beforeEach(nextFrame);
       beforeEach(updateComplete);
 
-
       it('should have visible scroll buttons if overflowed', function() {
         // Note: overflow buttons are not included in the accessibility tree otherwise we'd test
         // for buttons there. tabindex="-1" is used on the buttons to prevent focus and was a