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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix widgets overwrites from short to long-hand props #4935
Fix widgets overwrites from short to long-hand props #4935
Conversation
Input: { | ||
style: { | ||
padding: ".5rem", | ||
borderWidth: "1px", |
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.
No border here?
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.
Same as the other comment
}, | ||
}, | ||
|
||
InputContainer: { | ||
style: { | ||
borderWidth: "1px", |
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.
No border here?
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.
Yep, I removed that intentionally since it did not seem to actually do anything. Having borderWidth
in Root is the only place that has an impact.
馃摎 Context
The PR #4861 applied some changes to border and padding via short-hand properties (
padding
,borderWidth
) in the baseweb overwrites. Apparently, this leads to weird bugs and warnings:We need to use the long-hand properties instead (
borderLeftWidth
,borderRightWidth
,borderTopWidth
,borderBottomWidth
).What kind of change does this PR introduce?
馃 Description of Changes
馃И Testing Done
Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.