welldone-software/why-did-you-render

Wdyr ignores equality function in useSelector

Open

#190 opened on 2021年4月16日

GitHub で見る
 (3 comments) (1 reaction) (0 assignees)JavaScript (10,560 stars) (188 forks)batch import
Low prioritybuggood first issuehelp wanted

説明

When using wdyr I noticed that it reports unnecessary renders caused by useSelector with an equality function even though the equality function causes the render to be avoided. Looking at the code it looks like wdyr does not consider the result of the equality function.

An example:

useSelector(someSelector, (prev, next) => true);

コントリビューターガイド

技術スタック
javascriptreact
領域
frontenddeveloper experience
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
React basicsunderstanding of useSelectorfamiliarity with wdyr
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60
調査方針
Start by locating the code in the why did you render repository that tracks renders and detects unnecessary re renders. Specifically, look for how useSelector is patched and where the equality function is called. The issue suggests that the equality function's result is ignored. Investigate the patch logic in files like 'trackHooks.js' or similar. Check the existing comments and any linked PRs for additional context. Confirm that the equality function result is not considered and then implement a fix that respects it.
Wdyr ignores equality function in useSelector · welldone-software/why-did-you-render#190 | Good First Issue