{"action":"opened","issue":{"url":"https://api.github.com/repos/vaadin/flow/issues/4086","repository_url":"https://api.github.com/repos/vaadin/flow","labels_url":"https://api.github.com/repos/vaadin/flow/issues/4086/labels{/name}","comments_url":"https://api.github.com/repos/vaadin/flow/issues/4086/comments","events_url":"https://api.github.com/repos/vaadin/flow/issues/4086/events","html_url":"https://github.com/vaadin/flow/issues/4086","id":321067999,"number":4086,"title":"@ModelItem should be updated with the client side data","user":{"login":"stefanuebe","id":36725474,"avatar_url":"https://avatars3.githubusercontent.com/u/36725474?v=4","gravatar_id":"","url":"https://api.github.com/users/stefanuebe","html_url":"https://github.com/stefanuebe","followers_url":"https://api.github.com/users/stefanuebe/followers","following_url":"https://api.github.com/users/stefanuebe/following{/other_user}","gists_url":"https://api.github.com/users/stefanuebe/gists{/gist_id}","starred_url":"https://api.github.com/users/stefanuebe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stefanuebe/subscriptions","organizations_url":"https://api.github.com/users/stefanuebe/orgs","repos_url":"https://api.github.com/users/stefanuebe/repos","events_url":"https://api.github.com/users/stefanuebe/events{/privacy}","received_events_url":"https://api.github.com/users/stefanuebe/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-05-08T07:11:25Z","updated_at":"2018-05-08T07:11:25Z","closed_at":null,"author_association":"NONE","body":"I do not understand, what `@ModelItem` is thought of to be used for. In its current state it is not usable at all, since event doing something like `this.set(\"e.model.item.name\", \"e.currentTarget.value\");` on client side before sending the data back to the server does not update the model.\r\n\r\nSo the request is, that `@ModelItem` (or `@EventData(\"event.model.item\")`) should reflect the client side state to prevent the server side code being filled up with JS event data information like\r\n \r\n```\r\n@EventHandler\r\nprivate void handleOnChange(@EventData(\"event.currentTarget.id\") String id, @EventData(\"event.currentTaget.value\") newValue, ...) { ... }\r\n```\r\n\r\nsince this would corrupt the separation between client and server impl."}}
{"actor":{"display_login":"stefanuebe"}}