Strict TypeScript #2929 #2930
Strict TypeScript #2929 #2930
Conversation
Codecov Report
@@ Coverage Diff @@
## next #2930 +/- ##
==========================================
- Coverage 88.20% 83.01% -5.20%
==========================================
Files 50 50
Lines 1009 1048 +39
Branches 399 402 +3
==========================================
- Hits 890 870 -20
- Misses 76 137 +61
+ Partials 43 41 -2
Continue to review full report at Codecov.
|
src/buttons/ButtonGroup.tsx
Outdated
| verticalContainer: { | ||
| flexDirection: 'column', | ||
| height: null, | ||
| height: undefined, |
There was a problem hiding this comment.
This actually changes the behavior of the component. We should either turn off the null check in the tsconfig or find a different solution.
There was a problem hiding this comment.
I tried in demo app, it was working fine, but if it really results in some weired behaviour we can add "strictNullChecks": false in the end
There was a problem hiding this comment.
Yeah, it might just be a corner case, but I do know that width and height being set to null have a special meaning apart from undefined. At least for TS changes, it safest to try not to change any of the JS code. There might be a Stackoverflow though that talks about how to handle this situation.
There was a problem hiding this comment.
I will check for best solution
Uh oh!
There was an error while loading. Please reload this page.