Change from `null` to `undefined` introduces issues with `<value> !== undefined`
維護者通常 1 天內回覆
還沒有人認領這個 Issue。
評估
- 難度
- 4/5
- 預估耗時
- 3-5 天
- 新手友好度
- 48/100
- Issue 類型
- 缺陷
- 描述清晰度
- 基本清楚
- 活躍度
- 冷清
- 技術堆疊
- react, typescript
- 領域
- frontend
研究方向
Start with the TextField.tsx line linked in the issue, then follow the investigation in the linked comment and check Modal, Popover, Grid, and GridSpan, especially generateResponsiveCustomProperties. Review how each prop handles null and undefined, and decide whether defaulted props support null or need documentation. Done means resolving the listed checks and documenting the behavior decision.
由索引模型根據 Issue 內容生成。
描述
This is caused by https://github.com/react-ui-org/react-ui/issues/662
As result of mentioned change, TextField is incorrectly setting hasRootSmallInput class when inputSize is set to null. This is happening on following line: https://github.com/react-ui-org/react-ui/blob/71b2fe1cd5719128ebdf3d17b90f4b441ebe199d/src/components/TextField/TextField.tsx#L38
Problem is that if inputSize is set to null:
const hasSmallInput = (inputSize !== undefined) && (inputSize <= SMALL_INPUT_SIZE);
Then hasSmallInput is true because of inputSize <= SMALL_INPUT_SIZE => true. Why? Because null <= 10 ...
We need to investigate this and examine all occurrences of !== undefined and !== null as well. This is why I always recommend using != null to cover both cases.
Scope
Investigated in https://github.com/react-ui-org/react-ui/issues/673#issuecomment-5998585636.
- Use
!= nullinTextField,ModalandPopover(caused by #662) - Use
prop == nullingenerateResponsiveCustomProperties(Grid,GridSpan) - Decide whether
nullis supported for props with defaults (Popoverplacement,tag, …), or document that onlyundefinedmeans "not set"
- 主要語言
- TypeScript
- 星號
- 23
- 分支
- 10
- 平均合併
- 5 天 12 小時
- 30 天內合併 PR
- 9
環境準備
在瀏覽器裡用你自己的 GitHub 帳號啟動這個專案的開發容器。
- 沒有 Dockerfile 或 Docker Compose 檔案
- 沒有 Pull Request 範本
- 沒有貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
react-ui-org/react-ui 的其他 Issue
-
maintenance
難度 2/5 1-3 小時 新手友好度 78/100
react-ui-org/react-ui#784 ·
維護者通常 1 天內回覆
-
accessibility bug
難度 2/5 1-3 小時 新手友好度 68/100
react-ui-org/react-ui#775 ·
維護者通常 1 天內回覆
-
maintenance
難度 2/5 1-3 小時 新手友好度 63/100
react-ui-org/react-ui#753 · 1 則留言 ·
維護者通常 1 天內回覆
-
Playwright container fails `devEngines` check again because its image does not pin Node.js可能已有人在做 @bedrich-schindler 今天認領。 未關閉bug maintenance
難度 4/5 1-2 天 新手友好度 25/100
react-ui-org/react-ui#782 ·
維護者通常 1 天內回覆
-
maintenance
難度 4/5 3-5 天 新手友好度 42/100
react-ui-org/react-ui#780 ·
維護者通常 1 天內回覆
查看 react-ui-org/react-ui 的全部 Issue
相似的 Issue
-
unwrapDatabaseError masks real DB errors with TypeError: Cannot convert object to primitive value未關閉
難度 2/5 1-3 小時 新手友好度 74/100
NEARBuilders/everything-dev#798 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 78/100
vercel-labs/durable-harness#53 ·
維護者通常 1 天內回覆
-
[Bug]: Redesign tab bars ignore clicks on Linux: hidden chat layers keep window drag regions可能已有人在做 關聯的 PR 仍在進行中或已合併。 未關閉
難度 2/5 1-3 小時 新手友好度 82/100
維護者通常 1 天內回覆
-
experiência: a doc manda subir para o Micro do Supabase, mas não diz que exige plano pago nem quanto custa可能已有人在做 關聯的 PR 仍在進行中或已合併。 未關閉area/docs area/ui enhancement
難度 1/5 1-3 小時 新手友好度 88/100
melgarafael/DeskcommCRM#4098 ·
維護者通常 1 天內回覆
-
dependencies platform: ios
難度 1/5 1-3 小時 新手友好度 78/100
software-mansion/react-native-executorch#1617 ·
維護者通常 1 天內回覆