-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Set minimum width and height for views #25498
Conversation
This can help when creating custom ViewGroups
@M-i-k-e-l Shouldn't we drop in from |
@osdnk What do you mean by "drop in from |
Hello @M-i-k-e-l, thank you for the contribution. Could you please provide a test plan with an example when this might be useful? As far as I can see we already have minWidth and minHeight properties supported. |
Frankly speaking, I don't understand this PR. Can you add some text / gifs / screenshots / snack? |
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.
(min|max)(Width/Height)
(and all layout styles) must be consumed by Yoga only.
Summary: This sync includes the following changes: - **[54f297a60](facebook/react@54f297a60 )**: Enable useMemoCacheHook for ReactNative-fb build ([#25498](facebook/react#25498)) //<Jan Kassens>// Changelog: [General][Changed] - React Native sync for revisions 9fb581c...54f297a jest_e2e[run_all_tests] Reviewed By: poteto Differential Revision: D40435957 fbshipit-source-id: f447f50d80298e2c3e8be09ff52b1bedcfe6d402
Summary: This sync includes the following changes: - **[54f297a60](facebook/react@54f297a60 )**: Enable useMemoCacheHook for ReactNative-fb build ([facebook#25498](facebook/react#25498)) //<Jan Kassens>// Changelog: [General][Changed] - React Native sync for revisions 9fb581c...54f297a jest_e2e[run_all_tests] Reviewed By: poteto Differential Revision: D40435957 fbshipit-source-id: f447f50d80298e2c3e8be09ff52b1bedcfe6d402
Summary
This can help when creating custom ViewGroups (in native modules).
For example: images need to have size in order to be shown, and if the minimum size (width\height) is not set their size is 0.
Changelog
[Android] [Added] - add
setMinWidth
andsetMinHeight
toBaseViewManager
.[CATEGORY] [TYPE] - Message
Test Plan