{"action":"opened","issue":{"url":"https://api.github.com/repos/pfn/passifox/issues/351","labels_url":"https://api.github.com/repos/pfn/passifox/issues/351/labels{/name}","comments_url":"https://api.github.com/repos/pfn/passifox/issues/351/comments","events_url":"https://api.github.com/repos/pfn/passifox/issues/351/events","html_url":"https://github.com/pfn/passifox/issues/351","id":55379902,"number":351,"title":"Should trigger a change event when the password fields are filled in","user":{"login":"backspace","id":43280,"avatar_url":"https://avatars.githubusercontent.com/u/43280?v=3","gravatar_id":"","url":"https://api.github.com/users/backspace","html_url":"https://github.com/backspace","followers_url":"https://api.github.com/users/backspace/followers","following_url":"https://api.github.com/users/backspace/following{/other_user}","gists_url":"https://api.github.com/users/backspace/gists{/gist_id}","starred_url":"https://api.github.com/users/backspace/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/backspace/subscriptions","organizations_url":"https://api.github.com/users/backspace/orgs","repos_url":"https://api.github.com/users/backspace/repos","events_url":"https://api.github.com/users/backspace/events{/privacy}","received_events_url":"https://api.github.com/users/backspace/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-01-24T18:25:14Z","updated_at":"2015-01-24T18:25:14Z","closed_at":null,"body":"Thanks for your work on this project, it’s great to have an open source solution in the sea of proprietary ones.\r\n\r\nI was registering for a site and being rejected because it thought the password was blank, despite me having inserted one with chromeIPass. I did some investigating and found it’s because it’s a JavaScript application that uses two-way bindings to gather data from the `input` elements. When I ran `$('input[type=password]').change()` in the console, the application was able to pick up the password.\r\n\r\nIn the [source](https://github.com/pfn/passifox/blob/master/chromeipass/chromeipass.js#L300), you have this line:\r\n\r\n    field.val($password);\r\n\r\nIt would be sufficient to just change it to this:\r\n\r\n    field.val($password).change();\r\n\r\nI’d be happy to submit a pull request if it’s easier for you, but when I looked at the source I saw uses of `val` all over the place and wasn’t sure which ones were relevant. It should be safe to just apply it to all of them, but I thought I’d ask for feedback before going ahead with that."}}