Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Infra UI] Remove dependence on ResizeObserver polyfill #122022

Closed
Tracked by #121509
weltenwort opened this issue Dec 27, 2021 · 1 comment · Fixed by #148201
Closed
Tracked by #121509

[Infra UI] Remove dependence on ResizeObserver polyfill #122022

weltenwort opened this issue Dec 27, 2021 · 1 comment · Fixed by #148201
Assignees
@jennypavlova
Labels
Feature:Logs UI Logs UI feature Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@weltenwort
Copy link
Member

📓 Summary

Since support for IE11 has been dropped since 7.9.0, we should be able to rely on ResizeObserver being supported natively. That means we can reduce the bundle size by not importing the polyfill in the AutoSizer component anymore.

part of #121509

✔️ Acceptance criteria

  • The AutoSizer component uses the native ResizeObserver so that the infra bundle doesn't load or include a polyfill for it.
@weltenwort weltenwort added Feature:Metrics UI Metrics UI feature Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Dec 27, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

@weltenwort weltenwort mentioned this issue Dec 27, 2021
Open
5 tasks
@jennypavlova jennypavlova self-assigned this Dec 29, 2022
@jennypavlova jennypavlova mentioned this issue Dec 29, 2022
Merged
jennypavlova added a commit that referenced this issue Dec 30, 2022
@jennypavlova
[Infra UI] Remove dependence on ResizeObserver polyfill (#148201)
f1b8b4f 
Closes [#122022](#122022)
## Summary

This PR removes the import of `resize-observer-polyfill` in the
Autosizer component
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jennypavlova jennypavlova

Labels
Feature:Logs UI Logs UI feature Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Infra UI] Remove dependence on ResizeObserver polyfill jennypavlova/kibana
3 participants
@weltenwort @jennypavlova @elasticmachine