TJ (thaddeus jiang)@thaddeusjiangzh·19 May#CodeReviewLog 建议给没有文字的按钮添加 aria-label 收益: 1. 更好的 a11y 2. 容易编写测试代码 虽然之前被骂的很惨,但是还是分享一下吧。 人在做天在看,我是什么人,我自己知道。Çevir 中文60210
TJ (thaddeus jiang)@thaddeusjiangzh·17 MayReact 临时使用的 flag 变量,最好不要传递给其他 component,也不需要 interface。 #CodeReviewLogÇevir 中文00001
TJ (thaddeus jiang)@thaddeusjiangzh·16 MayReact 可能真的太难了,这个 bug 我们团队竟然没人能修复,我有点失望。 关键是我还教了好几遍。 Issue: 人材一覧页面中用户修改了 view 内容会显示 changed UI(小红点和 active save button),使用相同 URL 再次进入没有显示 changed UI 。 #CodeReviewLogÇevir 中文30603
TJ (thaddeus jiang)@thaddeusjiangzh·16 May请不要再把 && 当 if 使用了。 推荐阅读: kentcdodds.com/blog/use-terna… 我知道这可能是 React 官方文档的错误,但是不要管他们了,严于律己宽以待人。 bad code: #inline-if-with-logical--operator" target="_blank" rel="nofollow noopener">reactjs.org/docs/condition… #CodeReviewLogÇevir 中文744806
TJ (thaddeus jiang)@thaddeusjiangzh·9 May大家都知道 TypeScript any 是不好的,但是请注意 unknown 也不好。 #CodeReviewLogÇevir 中文2010
TJ (thaddeus jiang)@thaddeusjiangzh·28 Nis不需要这么特殊的高度,推荐使用 h-15 或者 h-16,可以心算出结果。 #CodeReviewLogÇevir 中文00101
TJ (thaddeus jiang)@thaddeusjiangzh·28 Nis如果你发现有特别多无用的 props ,要么是原来的 Component 写的不好,要么是你应该编写自己的 Component。 #CodeReviewLogÇevir 中文1080