welldone-software/why-did-you-render

Wdyr ignores equality function in useSelector

Open

#190 aberto em 16 de abr. de 2021

Ver no GitHub
 (3 comments) (1 reaction) (0 assignees)JavaScript (10.560 stars) (188 forks)batch import
Low prioritybuggood first issuehelp wanted

Description

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);

Guia do colaborador

Pilha de tecnologia
javascriptreact
Domain
frontenddeveloper experience
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
React basicsunderstanding of useSelectorfamiliarity with wdyr
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60
Direção de pesquisa
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.