Remove resize-observer-polyfill #230297
Merged
Remove resize-observer-polyfill #230297
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
legrego
added
chore
Team:Security
Contributor
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Module Count
Page load bundle
|
Contributor
|
Pinging @elastic/kibana-security (Team:Security) |
jbudz
reviewed
Aug 4, 2025
…kibana into remove-resize-observer-polyfill
jbudz
approved these changes
Aug 4, 2025
legrego
added
the
dependency-reduction
Contributor
|
Starting backport for target branches: 8.17, 8.18, 8.19, 9.1 https://github.com/elastic/kibana/actions/runs/16759172344 |
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Aug 5, 2025
close elastic#121509 This pull request removes the dependency on the `resize-observer-polyfill` package and replaces its TEST usage with a custom implementation of the `ResizeObserver` class. The changes simplify the codebase by eliminating the need for the polyfill and updating related test cases. ### Removal of `resize-observer-polyfill` dependency: * [`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357): Removed the `resize-observer-polyfill` package from the dependencies. ### Replacement with custom `ResizeObserver` implementation: * [`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45): Added a custom implementation of the `ResizeObserver` class to replace the polyfill. The custom class uses event listeners for `resize` events. ### Updates to test cases: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17): Removed the mock implementation of `resize-observer-polyfill` and its associated code, as it is no longer needed. [[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17) [[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53) ### Cleanup of polyfill usage in the main codebase: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12): Removed the import of `resize-observer-polyfill` from the `ResizeChecker` implementation. --------- Co-authored-by: Anton Dosov <anton.dosov@elastic.co> (cherry picked from commit 6bf697f)
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Aug 5, 2025
close elastic#121509 This pull request removes the dependency on the `resize-observer-polyfill` package and replaces its TEST usage with a custom implementation of the `ResizeObserver` class. The changes simplify the codebase by eliminating the need for the polyfill and updating related test cases. ### Removal of `resize-observer-polyfill` dependency: * [`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357): Removed the `resize-observer-polyfill` package from the dependencies. ### Replacement with custom `ResizeObserver` implementation: * [`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45): Added a custom implementation of the `ResizeObserver` class to replace the polyfill. The custom class uses event listeners for `resize` events. ### Updates to test cases: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17): Removed the mock implementation of `resize-observer-polyfill` and its associated code, as it is no longer needed. [[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17) [[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53) ### Cleanup of polyfill usage in the main codebase: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12): Removed the import of `resize-observer-polyfill` from the `ResizeChecker` implementation. --------- Co-authored-by: Anton Dosov <anton.dosov@elastic.co> (cherry picked from commit 6bf697f)
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Aug 5, 2025
close elastic#121509 This pull request removes the dependency on the `resize-observer-polyfill` package and replaces its TEST usage with a custom implementation of the `ResizeObserver` class. The changes simplify the codebase by eliminating the need for the polyfill and updating related test cases. ### Removal of `resize-observer-polyfill` dependency: * [`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357): Removed the `resize-observer-polyfill` package from the dependencies. ### Replacement with custom `ResizeObserver` implementation: * [`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45): Added a custom implementation of the `ResizeObserver` class to replace the polyfill. The custom class uses event listeners for `resize` events. ### Updates to test cases: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17): Removed the mock implementation of `resize-observer-polyfill` and its associated code, as it is no longer needed. [[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17) [[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53) ### Cleanup of polyfill usage in the main codebase: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12): Removed the import of `resize-observer-polyfill` from the `ResizeChecker` implementation. --------- Co-authored-by: Anton Dosov <anton.dosov@elastic.co> (cherry picked from commit 6bf697f)
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Aug 5, 2025
close elastic#121509 This pull request removes the dependency on the `resize-observer-polyfill` package and replaces its TEST usage with a custom implementation of the `ResizeObserver` class. The changes simplify the codebase by eliminating the need for the polyfill and updating related test cases. ### Removal of `resize-observer-polyfill` dependency: * [`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357): Removed the `resize-observer-polyfill` package from the dependencies. ### Replacement with custom `ResizeObserver` implementation: * [`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45): Added a custom implementation of the `ResizeObserver` class to replace the polyfill. The custom class uses event listeners for `resize` events. ### Updates to test cases: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17): Removed the mock implementation of `resize-observer-polyfill` and its associated code, as it is no longer needed. [[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17) [[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53) ### Cleanup of polyfill usage in the main codebase: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12): Removed the import of `resize-observer-polyfill` from the `ResizeChecker` implementation. --------- Co-authored-by: Anton Dosov <anton.dosov@elastic.co> (cherry picked from commit 6bf697f)
Contributor
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
kibanamachine
added a commit
that referenced
this pull request
Aug 5, 2025
# Backport This will backport the following commits from `main` to `8.19`: - [Remove resize-observer-polyfill (#230297)](#230297) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Larry Gregory","email":"larry.gregory@elastic.co"},"sourceCommit":{"committedDate":"2025-08-05T19:17:43Z","message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["chore","Team:Security","release_note:skip","backport:prev-minor","backport:prev-major","v9.2.0","dependency-reduction"],"title":"Remove resize-observer-polyfill","number":230297,"url":"https://github.com/elastic/kibana/pull/230297","mergeCommit":{"message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/230297","number":230297,"mergeCommit":{"message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d"}}]}] BACKPORT--> Co-authored-by: Larry Gregory <larry.gregory@elastic.co> Co-authored-by: Anton Dosov <anton.dosov@elastic.co>
kibanamachine
added a commit
that referenced
this pull request
Aug 5, 2025
# Backport This will backport the following commits from `main` to `9.1`: - [Remove resize-observer-polyfill (#230297)](#230297) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Larry Gregory","email":"larry.gregory@elastic.co"},"sourceCommit":{"committedDate":"2025-08-05T19:17:43Z","message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["chore","Team:Security","release_note:skip","backport:prev-minor","backport:prev-major","v9.2.0","dependency-reduction"],"title":"Remove resize-observer-polyfill","number":230297,"url":"https://github.com/elastic/kibana/pull/230297","mergeCommit":{"message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/230297","number":230297,"mergeCommit":{"message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d"}}]}] BACKPORT--> Co-authored-by: Larry Gregory <larry.gregory@elastic.co> Co-authored-by: Anton Dosov <anton.dosov@elastic.co>
kibanamachine
added a commit
that referenced
this pull request
Aug 5, 2025
# Backport This will backport the following commits from `main` to `8.17`: - [Remove resize-observer-polyfill (#230297)](#230297) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Larry Gregory","email":"larry.gregory@elastic.co"},"sourceCommit":{"committedDate":"2025-08-05T19:17:43Z","message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["chore","Team:Security","release_note:skip","backport:prev-minor","backport:prev-major","v9.2.0","dependency-reduction"],"title":"Remove resize-observer-polyfill","number":230297,"url":"https://github.com/elastic/kibana/pull/230297","mergeCommit":{"message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/230297","number":230297,"mergeCommit":{"message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d"}}]}] BACKPORT--> Co-authored-by: Larry Gregory <larry.gregory@elastic.co> Co-authored-by: Anton Dosov <anton.dosov@elastic.co>
kibanamachine
added a commit
that referenced
this pull request
Aug 5, 2025
# Backport This will backport the following commits from `main` to `8.18`: - [Remove resize-observer-polyfill (#230297)](#230297) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Larry Gregory","email":"larry.gregory@elastic.co"},"sourceCommit":{"committedDate":"2025-08-05T19:17:43Z","message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["chore","Team:Security","release_note:skip","backport:prev-minor","backport:prev-major","v9.2.0","dependency-reduction"],"title":"Remove resize-observer-polyfill","number":230297,"url":"https://github.com/elastic/kibana/pull/230297","mergeCommit":{"message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/230297","number":230297,"mergeCommit":{"message":"Remove resize-observer-polyfill (#230297)\n\nclose https://github.com/elastic/kibana/issues/121509\n\nThis pull request removes the dependency on the\n`resize-observer-polyfill` package and replaces its TEST usage with a\ncustom implementation of the `ResizeObserver` class. The changes\nsimplify the codebase by eliminating the need for the polyfill and\nupdating related test cases.\n\n### Removal of `resize-observer-polyfill` dependency:\n\n*\n[`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357):\nRemoved the `resize-observer-polyfill` package from the dependencies.\n\n### Replacement with custom `ResizeObserver` implementation:\n\n*\n[`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45):\nAdded a custom implementation of the `ResizeObserver` class to replace\nthe polyfill. The custom class uses event listeners for `resize` events.\n\n### Updates to test cases:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17):\nRemoved the mock implementation of `resize-observer-polyfill` and its\nassociated code, as it is no longer needed.\n[[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17)\n[[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53)\n\n### Cleanup of polyfill usage in the main codebase:\n\n*\n[`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12):\nRemoved the import of `resize-observer-polyfill` from the\n`ResizeChecker` implementation.\n\n---------\n\nCo-authored-by: Anton Dosov <anton.dosov@elastic.co>","sha":"6bf697f308628b6c894896a94276b5f43cc8dd3d"}}]}] BACKPORT--> Co-authored-by: Larry Gregory <larry.gregory@elastic.co> Co-authored-by: Anton Dosov <anton.dosov@elastic.co>
delanni
pushed a commit
to delanni/kibana
that referenced
this pull request
Aug 5, 2025
close elastic#121509 This pull request removes the dependency on the `resize-observer-polyfill` package and replaces its TEST usage with a custom implementation of the `ResizeObserver` class. The changes simplify the codebase by eliminating the need for the polyfill and updating related test cases. ### Removal of `resize-observer-polyfill` dependency: * [`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357): Removed the `resize-observer-polyfill` package from the dependencies. ### Replacement with custom `ResizeObserver` implementation: * [`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45): Added a custom implementation of the `ResizeObserver` class to replace the polyfill. The custom class uses event listeners for `resize` events. ### Updates to test cases: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17): Removed the mock implementation of `resize-observer-polyfill` and its associated code, as it is no longer needed. [[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17) [[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53) ### Cleanup of polyfill usage in the main codebase: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12): Removed the import of `resize-observer-polyfill` from the `ResizeChecker` implementation. --------- Co-authored-by: Anton Dosov <anton.dosov@elastic.co>
NicholasPeretti
pushed a commit
to NicholasPeretti/kibana
that referenced
this pull request
Aug 18, 2025
close elastic#121509 This pull request removes the dependency on the `resize-observer-polyfill` package and replaces its TEST usage with a custom implementation of the `ResizeObserver` class. The changes simplify the codebase by eliminating the need for the polyfill and updating related test cases. ### Removal of `resize-observer-polyfill` dependency: * [`package.json`](diffhunk://#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L1357): Removed the `resize-observer-polyfill` package from the dependencies. ### Replacement with custom `ResizeObserver` implementation: * [`src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js`](diffhunk://#diff-d20f940201d6ffb3751d4696a2502f77aebcd9cbbb71c38e65ac2bc76516ed57L34-R45): Added a custom implementation of the `ResizeObserver` class to replace the polyfill. The custom class uses event listeners for `resize` events. ### Updates to test cases: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts`](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17): Removed the mock implementation of `resize-observer-polyfill` and its associated code, as it is no longer needed. [[1]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL13-L17) [[2]](diffhunk://#diff-0aa7cb2b3833f42456fdf78b791624b2a2a503115a5762f3b9fc60fe9ec3269dL44-L53) ### Cleanup of polyfill usage in the main codebase: * [`src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts`](diffhunk://#diff-c28fee2aa7f94e97473c034328cdc2d9e951627be00cc5978332993133d0a899L12): Removed the import of `resize-observer-polyfill` from the `ResizeChecker` implementation. --------- Co-authored-by: Anton Dosov <anton.dosov@elastic.co>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
close #121509
This pull request removes the dependency on the
resize-observer-polyfillpackage and replaces its TEST usage with a custom implementation of theResizeObserverclass. The changes simplify the codebase by eliminating the need for the polyfill and updating related test cases.Removal of
resize-observer-polyfilldependency:package.json: Removed theresize-observer-polyfillpackage from the dependencies.Replacement with custom
ResizeObserverimplementation:src/platform/packages/shared/kbn-test/src/jest/setup/polyfills.jsdom.js: Added a custom implementation of theResizeObserverclass to replace the polyfill. The custom class uses event listeners forresizeevents.Updates to test cases:
src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.test.ts: Removed the mock implementation ofresize-observer-polyfilland its associated code, as it is no longer needed. [1] [2]Cleanup of polyfill usage in the main codebase:
src/platform/plugins/shared/kibana_utils/public/resize_checker/resize_checker.ts: Removed the import ofresize-observer-polyfillfrom theResizeCheckerimplementation.