Conversation
Size changesDetails📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
alioguzhan
left a comment
There was a problem hiding this comment.
Elinize saglik. 2 nokta var sadece. Sonrasinda merge edelim.
| --- | ||
|
|
||
| ### Optimizing re-rendering on every keystroke {/*optimizing-re-rendering-on-every-keystroke*/} | ||
| ### Yeniden renderın klavyeye her tıklandığında en iyi hale getirilmesi {/*optimizing-re-rendering-on-every-keystroke*/} |
There was a problem hiding this comment.
- Her tuş basımında yeniden render işlemininin iyileştirilmesi
Bu daha iyi gibi sanki. basımında yerine vuruşunda da kullanılabilir.
| Önce `value={undefined}` olarak değer iletip daha sonra `value="some string"` şeklinde değer iletemezsiniz çünkü React, kontrollü bileşen mi kontrolsüz bileşen mi bunu bilemez. Kontrollü bileşen her zaman `value` değer olarak string almalıdır, `null` ya da `undefined` almamalıdır. | ||
|
|
||
| Similarly, if you pass `checked` to a checkbox, ensure it's always a boolean. | ||
| `value` değeriniz API'den veya durum değişkeninden geliyorsa, başta `null` veya `undefined` olarak tanımlanmış olabilir. Bu durumda boş bir string (`''`) olarak tanımlayabilirsiniz veya `value={someValue == ''}` olarak iletip `value` değerinin string olup olmadığına emin olabilirsiniz. |
There was a problem hiding this comment.
Buradaki someValue ?? '' kodunu değiştirmişsiniz sanırım. someValue == '' doğru şekilde çalışmaz.
Yanlışlıkla öyle yazıldığını düşündünüz sanırım. ?? geçerli bir operator. (Nullish coalescing operator).
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Nullish_coalescing
There was a problem hiding this comment.
Öncelikle geri dönütünüz için teşekkür ederim. Burada ?? yerine yanlışlıkla == kullanmışım, diğer yorumunuzla birlikte burayı da hemen düzeltiyorum.
alioguzhan
left a comment
There was a problem hiding this comment.
Bitirdiginizde tekrar review talep ederseniz hemen incelerim.
alioguzhan
left a comment
There was a problem hiding this comment.
Elinize saglik. Tesekkurler. Merge ediyorum.
No description provided.