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.

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

deps: replace resize-observer-polyfill with @juggle/resize-observer #1422

Merged
merged 2 commits into from
Jan 26, 2022

Conversation

williaster
Copy link
Collaborator

🏠 Internal

Fixes #1325 . This refactors our usage of resize-observer-polyfill to @juggle/resize-observer. The former is no longer maintained and its ResizeObserver types will conflict with typescript@^4's own type defs (see que-etc/resize-observer-polyfill#83, thus this fixes one subtask of #1421).

It makes the changes in the following packages:

  • @visx/responsive: dependency
  • @visx/annotation: devDependency
  • @visx/xychart: devDependency
  • @visx/tooltip: devDependency (this was actually missing previously)

@gazcn007 @kristw @hshoff
cc @gbiryukov

@github-actions
Copy link

Size Changes

Package Diff ESM Prev ESM CJS Prev CJS
visx-responsive +0.0% 22.05 KB 22.05 KB 27.31 KB 27.37 KB

Compared to master. File sizes are unminified and ungzipped.

View raw build stats

Previous (master)

{
  "visx-annotation": {
    "esm": 29031,
    "lib": 40228
  },
  "visx-axis": {
    "esm": 20859,
    "lib": 25340
  },
  "visx-bounds": {
    "esm": 3028,
    "lib": 3450
  },
  "visx-brush": {
    "esm": 56089,
    "lib": 60231
  },
  "visx-chord": {
    "esm": 3486,
    "lib": 4716
  },
  "visx-clip-path": {
    "esm": 4502,
    "lib": 6062
  },
  "visx-curve": {
    "esm": 323,
    "lib": 1464
  },
  "visx-demo": {
    "esm": 0,
    "lib": 0
  },
  "visx-drag": {
    "esm": 12834,
    "lib": 16829
  },
  "visx-event": {
    "esm": 3807,
    "lib": 5200
  },
  "visx-geo": {
    "esm": 13425,
    "lib": 16705
  },
  "visx-glyph": {
    "esm": 15109,
    "lib": 20013
  },
  "visx-gradient": {
    "esm": 18135,
    "lib": 22853
  },
  "visx-grid": {
    "esm": 19057,
    "lib": 22924
  },
  "visx-group": {
    "esm": 1646,
    "lib": 2274
  },
  "visx-heatmap": {
    "esm": 7414,
    "lib": 8770
  },
  "visx-hierarchy": {
    "esm": 12209,
    "lib": 18111
  },
  "visx-legend": {
    "esm": 27079,
    "lib": 34279
  },
  "visx-marker": {
    "esm": 9101,
    "lib": 11337
  },
  "visx-mock-data": {
    "esm": 326005,
    "lib": 329416
  },
  "visx-network": {
    "esm": 4690,
    "lib": 6869
  },
  "visx-pattern": {
    "esm": 11826,
    "lib": 15957
  },
  "visx-point": {
    "esm": 1266,
    "lib": 2095
  },
  "visx-react-spring": {
    "esm": 13780,
    "lib": 18784
  },
  "visx-responsive": {
    "esm": 22577,
    "lib": 28031
  },
  "visx-scale": {
    "esm": 18512,
    "lib": 30389
  },
  "visx-shape": {
    "esm": 87206,
    "lib": 109413
  },
  "visx-stats": {
    "esm": 13830,
    "lib": 15432
  },
  "visx-text": {
    "esm": 8574,
    "lib": 11304
  },
  "visx-threshold": {
    "esm": 2911,
    "lib": 3820
  },
  "visx-tooltip": {
    "esm": 14746,
    "lib": 21787
  },
  "visx-visx": {
    "esm": 1467,
    "lib": 4429
  },
  "visx-voronoi": {
    "esm": 2313,
    "lib": 3033
  },
  "visx-wordcloud": {
    "esm": 2661,
    "lib": 4673
  },
  "visx-xychart": {
    "esm": 174959,
    "lib": 249292
  },
  "visx-zoom": {
    "esm": 16099,
    "lib": 19179
  }
}

Current

{
  "visx-annotation": {
    "esm": 29031,
    "lib": 40228
  },
  "visx-axis": {
    "esm": 20859,
    "lib": 25340
  },
  "visx-bounds": {
    "esm": 3028,
    "lib": 3450
  },
  "visx-brush": {
    "esm": 56089,
    "lib": 60231
  },
  "visx-chord": {
    "esm": 3486,
    "lib": 4716
  },
  "visx-clip-path": {
    "esm": 4502,
    "lib": 6062
  },
  "visx-curve": {
    "esm": 323,
    "lib": 1464
  },
  "visx-demo": {
    "esm": 0,
    "lib": 0
  },
  "visx-drag": {
    "esm": 12834,
    "lib": 16829
  },
  "visx-event": {
    "esm": 3807,
    "lib": 5200
  },
  "visx-geo": {
    "esm": 13425,
    "lib": 16705
  },
  "visx-glyph": {
    "esm": 15109,
    "lib": 20013
  },
  "visx-gradient": {
    "esm": 18135,
    "lib": 22853
  },
  "visx-grid": {
    "esm": 19057,
    "lib": 22924
  },
  "visx-group": {
    "esm": 1646,
    "lib": 2274
  },
  "visx-heatmap": {
    "esm": 7414,
    "lib": 8770
  },
  "visx-hierarchy": {
    "esm": 12209,
    "lib": 18111
  },
  "visx-legend": {
    "esm": 27079,
    "lib": 34279
  },
  "visx-marker": {
    "esm": 9101,
    "lib": 11337
  },
  "visx-mock-data": {
    "esm": 326005,
    "lib": 329416
  },
  "visx-network": {
    "esm": 4690,
    "lib": 6869
  },
  "visx-pattern": {
    "esm": 11826,
    "lib": 15957
  },
  "visx-point": {
    "esm": 1266,
    "lib": 2095
  },
  "visx-react-spring": {
    "esm": 13780,
    "lib": 18784
  },
  "visx-responsive": {
    "esm": 22583,
    "lib": 27963
  },
  "visx-scale": {
    "esm": 18512,
    "lib": 30389
  },
  "visx-shape": {
    "esm": 87206,
    "lib": 109413
  },
  "visx-stats": {
    "esm": 13830,
    "lib": 15432
  },
  "visx-text": {
    "esm": 8574,
    "lib": 11304
  },
  "visx-threshold": {
    "esm": 2911,
    "lib": 3820
  },
  "visx-tooltip": {
    "esm": 14746,
    "lib": 21787
  },
  "visx-visx": {
    "esm": 1467,
    "lib": 4429
  },
  "visx-voronoi": {
    "esm": 2313,
    "lib": 3033
  },
  "visx-wordcloud": {
    "esm": 2661,
    "lib": 4673
  },
  "visx-xychart": {
    "esm": 174959,
    "lib": 249292
  },
  "visx-zoom": {
    "esm": 16099,
    "lib": 19179
  }
}

@williaster williaster merged commit 4c71be5 into master Jan 26, 2022
@williaster williaster deleted the chris--new-resize-obs branch January 26, 2022 01:21
@github-actions
Copy link

🎉 This PR is included in version v2.8.0 of the packages modified 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider replacing resize-observer-polyfill
2 participants