tleunen/react-mdl

Menu items get broken after update

Open

#400 ouverte le 19 sept. 2016

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)CSS (1 772 stars) (259 forks)batch import
bughelp wanted

Description

I've created component that renders Menu with 3 items ['a', 'b', 'c'] from state. That works fine. Then I update component state items to ['a', 'x', 'y']. Component re-renders menu, but updated items are broken - they does not hide after click.

Please have a look on codepen for that: http://codepen.io/anon/pen/gwwKvk?editors=0011

Guide contributeur

Stack technique
javascriptreactcss
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
basic React
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Review the Menu component implementation in react mdl to understand how items are rendered and event handlers are attached. The bug likely involves React's key reconciliation or state update not properly reinitializing menu item click handlers. Check the codepen example to reproduce the issue. Focus on the component's shouldComponentUpdate or componentDidUpdate lifecycle methods.