{"comment":{"url":"https://api.github.com/repos/schuel/hmmm/comments/16979879","html_url":"https://github.com/schuel/hmmm/commit/75063a461c865c6cac4364284a87db17a88e2fa2#commitcomment-16979879","id":16979879,"user":{"login":"sbalmer","id":1814660,"avatar_url":"https://avatars.githubusercontent.com/u/1814660?v=3","gravatar_id":"","url":"https://api.github.com/users/sbalmer","html_url":"https://github.com/sbalmer","followers_url":"https://api.github.com/users/sbalmer/followers","following_url":"https://api.github.com/users/sbalmer/following{/other_user}","gists_url":"https://api.github.com/users/sbalmer/gists{/gist_id}","starred_url":"https://api.github.com/users/sbalmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sbalmer/subscriptions","organizations_url":"https://api.github.com/users/sbalmer/orgs","repos_url":"https://api.github.com/users/sbalmer/repos","events_url":"https://api.github.com/users/sbalmer/events{/privacy}","received_events_url":"https://api.github.com/users/sbalmer/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"75063a461c865c6cac4364284a87db17a88e2fa2","created_at":"2016-04-05T22:51:18Z","updated_at":"2016-04-05T22:51:18Z","body":"Those <form> tags are necessary for the browser so it knows the buttons belong to the input fields. If you remove them, the browser doesn't know it should submit on pressing enter, and that's just one of the effects. In this case, even clicking the event saving button did not work anymore because the code listened for 'submit' events, not 'click' events.\r\n\r\nI understand that you want to clean out unnecessary tags and I welcome that! Removing cruft is an important part of a healthy codebase. In this case you removed tags with semantic meaning though. I fixed the two places (event editing and login form) where I saw the form tags removed. Please re-add them if they are missing somewhere else now."}}