Semantic-Org/Semantic-UI-React

Popup: triggers that do not spread props do not work

Open

#1.187 geöffnet am 19. Jan. 2017

Auf GitHub ansehen
 (8 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (13.136 Stars) (4.065 Forks)batch import
enhancementhelp wanted

Beschreibung

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

Contributor Guide

Tech Stack
reactjavascript
Domain
frontend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
React basicsunderstanding of component props
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
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.