Semantic-Org/Semantic-UI-React

Popup: triggers that do not spread props do not work

Open

#1 187 ouverte le 19 janv. 2017

Voir sur GitHub
 (8 commentaires) (1 réaction) (0 assignés)JavaScript (13 136 stars) (4 065 forks)batch import
enhancementhelp wanted

Description

Steps

  1. Create a stateless component.
  2. Pass it to Popup as the trigger prop.

Expected Result

Get the popup when mousing over the trigger component.

Actual Result

Don't get a popup when mousing over the trigger component.

Version

0.64.2 (master)

Testcase

http://codepen.io/anon/pen/VPpZQL

Guide contributeur

Stack technique
reactjavascript
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
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.
clear
Prérequis
React basicsunderstanding of component props
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
The bug occurs when a stateless component is passed as the `trigger` prop without spreading props. Investigate the Popup component's source code (likely in `src/modules/Popup`) to see how it renders the trigger. Check if there is a missing spread of props onto the trigger element. The provided CodePen testcase can be used to replicate. Look at how similar components handle prop spreading in this codebase.