Set value by keyboard (#81)

Moved setValue to input's onChange from the label's onClick. This allows the user to set the value with either their keyboard or with their mouse.
lorensr · Jan 17, 2023 · feb07bc · feb07bc
1 parent af172c0
commit feb07bc
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 9 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/SegmentedControlWithoutStyles.js b/src/SegmentedControlWithoutStyles.js
@@ -46,12 +46,12 @@ class SegmentedControlWithoutStyles extends Component {
             id={getId(option)}
             defaultChecked={option.default}
             disabled={option.disabled}
+            onChange={() => this.setValue(option.value)}
           />
         ))}
         {this.props.options.map((option) => (
           <label
             key={option.value}
-            onClick={() => this.setValue(option.value)}
             htmlFor={getId(option)}
             data-value={option.label}
           >