helpscout / blue / 61

100%

master: 100%

push

travis-ci

Choice: Fix ID generation

This update fixes the `uniqueID()` generation for the `Choice` component, which indirectly fixes it for `Checkbox` and `Radio`. The same fix was also applied to `Input` and `PortalWrapper`.

This issue was the element's ID was being regenerated everytime there was a prop change, which is incorrect. It should only be generated once. The solution was to generate and store the uniqueID in the component state, instead of creating it in the render hook.

The uniqueID creation factory function has also been enhanced to accept a prefix override namespace.

345 of 345 branches covered (100.0%)

Branch coverage included in aggregate %.

11 of 11 new or added lines in 5 files covered. (100.0%)

564 of 564 relevant lines covered (100.0%)

35.43 hits per line