InspectorControls.ToggleControl onChange callback is passed whole event object as argument

[shramee]

Issue Overview

ToggleControl of block InspectorControls passes event object as parameter to onChange. Whereas other InspectorControls controls pass value of the control.

Steps to Reproduce (for bugs)

1. Add ToggleControls and any other control e.g. RangeControl
2. Make onChange callback console log the argument received.
3. Compare ToggleControl and Range control outputs.

Expected Behavior

onChange callback should be passed event.target.checked as argument (Other controls pass event.target.value).

Current Behavior

Whole event object is passed which is not consistent with other controls.

Possible Solution

Well... Will try to do a PR... Idea is to do an anonymous function that calls onChange callback with checked property of the event target.

Screenshots / Video

Todos

• Tests
• Documentation

[jordymeow]

I confirm this issue with Gutenberg 2.1.0.