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
[React] fix missing key error #4193
Conversation
@@ -79,7 +79,7 @@ export default function positionChildrenUnderViews({children, viewports, deck, C | |||
deck._onViewStateChange(params); | |||
} | |||
}; | |||
return createElement(ContextProvider, {value: contextValue}, viewElement); | |||
return createElement(ContextProvider, {key: 'context', value: contextValue}, viewElement); |
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.
This key is not unique.
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.
- Run
yarn lint fix
to apply prettier. - You can just use the
key
variable in this function.
bd74472
to
44fda55
Compare
Pull request visgl#4098 partially fixed issue with missing key error. Context element is used in array children and also misses key. This PR fully fixes issue visgl#4116.
44fda55
to
f159bfd
Compare
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.
Thank you!
* master: (82 commits) fix typo in performance.md pydeck: ArcLayer, BitmapLayer, ColumnLayer examples (#4189) [React] fix missing key error (#4193) [Bug] Fix hexagon layer projection (#4173) Remove HtmlWebpackPlugin from examples/playground (#4178) @deck.gl/json: Fix bug dropping props with falsy values (#4185) Fix buffer size check in Attribute.updateBuffer (#4190) Bump luma dependency (#4191) data-filter: support double precision (#4163) Use int type for enum uniforms (#4171) [TileLayer] fix tile indices generation in edge cases (#4170) v8.1.0-alpha.1 Voodoo fix for Mac+NVIDIA bug (#4166) Remove unnecessary code from project glsl (#4162) Fix H3HexagonLayer update when viewport jumps (#4158) Refactor render tests; use stricter pass criteria (#4157) [Extension] Add source_target to brushing mode (#4150) Add offset feature to PathStyleExtension (#4126) Project module: support pre-projected positions (#4140) Repeat maps at low zoom levels (#4105) ...
Pull request #4098 partially fixed issue with missing key error. Context
element is used in array children and also misses key. This PR fully fixes
issue #4116.