This repository has been archived by the owner on May 9, 2018. It is now read-only.

Update example of targetChecked #108

Open

fluxxu wants to merge 1 commit into evancz:master from fluxxu:master

fluxxu commented Mar 29, 2016

I think it is better to use 'change' instead of 'input' because 'input' never fires for checkbox.

Tested in:
Chrome 49.0.2623.87 (64-bit)
Safari 9.0.3 (11601.4.4)
Firefox 44.0.2

Test Code:

<input type="checkbox" id="t"/>

document.getElementById('t').addEventListener('input', function () {
    alert('input')
})


          Update example of targetChecked

469a36c

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet