-
Notifications
You must be signed in to change notification settings - Fork 67
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
Release react 5.0.0-alpha.0 #214
Conversation
+ To aid in merge conflict resolution
* 5.0.0-wip cherry-pick from 5.1.0-wip — CPLAT-357: React 16 from NPM and Bundled with Webpack # Conflicts: # lib/react_client/react_interop.dart # lib/react_test_utils.dart
* 5.0.0-wip cherry-pick from 5.1.0-wip — testEvent logic could create false positives # Conflicts: # pubspec.yaml
Cplat 4310 pointer events
CPLAT-4335 Add deprecations in preparation for 5.0.0 # Conflicts: # lib/react.dart # lib/react_client/react_interop.dart # lib/react_test_utils.dart
CPLAT-4789: fix string refs # Conflicts: # lib/react_dom.js # lib/react_dom.js.map # lib/react_dom_prod.js # lib/react_dom_prod.js.map # lib/react_dom_server.js # lib/react_dom_server.js.map # lib/react_dom_server_prod.js # lib/react_dom_server_prod.js.map # test/factory/common_factory_tests.dart # test/factory/dart_factory_test.dart # test/factory/dom_factory_test.dart
Moving things around inevitably creates issues
- omit types on intializing formals - update package.json - update webpack - remove unused dart-helpers.js - rebuild bundles - remove out-dated tests # Conflicts: # lib/react.dart # lib/react.js.map # lib/react_client.dart # lib/react_client/js_interop_helpers.dart # lib/react_client/react_interop.dart # lib/react_prod.js # lib/react_prod.js.map # lib/react_test_utils.dart # lib/react_with_addons.js.map # lib/react_with_react_dom_prod.js # lib/react_with_react_dom_prod.js.map # package-lock.json # package.json # test/js_interop_helpers_test/shared_tests.dart # test/util.dart
+ To prevent the dartfmt command from producing different output using the Dart 1 SDK vs. the Dart 2 SDK during CI
# Conflicts: # lib/react_client.dart
+ Do not show in IDE analyzer view + Also seen in over_react: Workiva/over_react@3eceba3
Add bindings for React Portals
# Conflicts: # pubspec.yaml
… clean up implementations
Fix eventdata
/// [children] can be any renderable React child, such as a [ReactElement], [String], or fragment. | ||
/// | ||
/// See: <https://reactjs.org/docs/portals.html> | ||
static ReactPortal createPortal(dynamic children, Element container) => ReactDOM.createPortal(children, container); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@greglittlefield-wf @joebingham-wk @kealjones-wk just confirming that this is an intentional addition in a major release instead of waiting for the subsequent minor?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's right, so that consumers (like WSD) are able to transition from react_dom.render
to portals.
+1, merging |
ReactJS 16.x Support
.js
files provided by this package are now ReactJS version 16.5.1.0
.FYA: @greglittlefield-wf @kealjones-wk @joebingham-wk @sydneyjodon-wk