Skip to content

Conversation

nwidynski
Copy link
Contributor

@nwidynski nwidynski commented Oct 9, 2025

This reverts #2860 to reduce bundle sizes and improve runtime performance. This change was originally scheduled for July 2022 so this should not come unexpected by now 😅

Noteworthy: Chrome 85+ addressed scrollLeft RTL compatibility issues, currently normalized by our virtualization utils, but simultaneously introduced new direction issues with scrollTop 🫠

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

@snowystinger
Copy link
Member

We'll need to make sure that this isn't still the case #8384 (comment)
as well as making sure this still works #7137

I can make a PR to turn on our verdaccio based fake "prod" build soon

@nwidynski
Copy link
Contributor Author

@snowystinger I linked Devons original comment for the planned deprecation date of Webpack 4 (July 2022). Has this been extended in the meantime?

@snowystinger
Copy link
Member

snowystinger commented Oct 9, 2025

@snowystinger I linked Devons original comment for the planned deprecation date of Webpack 4 (July 2022). Has this been extended in the meantime?

Yeah, I noticed that link after I posted, thanks for that.

Unfortunately it's not about deprecation of webpack 4, it's about usage. A brief glance at https://www.npmjs.com/package/webpack/v/5.102.1?activeTab=versions would seem to indicate that the usage of webpack 5 has grown a lot.

Could probably add up the downloads across the major versions to get a statistic.

Somewhat specifically, we'll have to check inside Adobe as well...

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

Successfully merging this pull request may close these issues.

2 participants