Semantic-Org/Semantic-UI-React

Radio: onChange not triggered for radio toggle with id

Open

Aperta il 6 ago 2019

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)JavaScript (13.136 star) (4065 fork)batch import
bughelp wanted

Descrizione

Bug Report

onChange not triggered on Radio with toggle and id props. And does okay without id or with name props.

Steps

  1. Add id and toggle props to Radio component
  2. Click on radio element

Expected Result

onChange should be triggered.

Actual Result

onChange is not triggered.

Version

0.87.3

Testcase

https://codesandbox.io/embed/semantic-ui-react-example-yowvx

Guida contributor

Tech stack
javascriptreact
Dominio
frontend
Tipo issue
bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
stale
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
React basicsSemantic UI React usage
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80
Direzione di ricerca
Investigate the Radio component's source code in Semantic UI React, focusing on the event handler when both 'id' and 'toggle' props are provided. Check the component's onChange implementation and see if there is a conflict with the 'id' attribute. Reproduce the bug using the provided CodeSandbox. Look at existing tests for Radio to understand expected behavior and consider adding a test for this specific case.