{"action":"created","issue":{"url":"https://api.github.com/repos/AmpersandJS/ampersand-select-view/issues/24","labels_url":"https://api.github.com/repos/AmpersandJS/ampersand-select-view/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/AmpersandJS/ampersand-select-view/issues/24/comments","events_url":"https://api.github.com/repos/AmpersandJS/ampersand-select-view/issues/24/events","html_url":"https://github.com/AmpersandJS/ampersand-select-view/pull/24","id":57395515,"number":24,"title":"ADDED: Don't validate until the select is either blurred, or the user tr...","user":{"login":"erd0s","id":2149743,"avatar_url":"https://avatars.githubusercontent.com/u/2149743?v=3","gravatar_id":"","url":"https://api.github.com/users/erd0s","html_url":"https://github.com/erd0s","followers_url":"https://api.github.com/users/erd0s/followers","following_url":"https://api.github.com/users/erd0s/following{/other_user}","gists_url":"https://api.github.com/users/erd0s/gists{/gist_id}","starred_url":"https://api.github.com/users/erd0s/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/erd0s/subscriptions","organizations_url":"https://api.github.com/users/erd0s/orgs","repos_url":"https://api.github.com/users/erd0s/repos","events_url":"https://api.github.com/users/erd0s/events{/privacy}","received_events_url":"https://api.github.com/users/erd0s/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/AmpersandJS/ampersand-select-view/labels/enhancement","name":"enhancement","color":"207de5"},{"url":"https://api.github.com/repos/AmpersandJS/ampersand-select-view/labels/request","name":"request","color":"c7def8"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":14,"created_at":"2015-02-11T23:41:56Z","updated_at":"2015-04-12T07:43:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/AmpersandJS/ampersand-select-view/pulls/24","html_url":"https://github.com/AmpersandJS/ampersand-select-view/pull/24","diff_url":"https://github.com/AmpersandJS/ampersand-select-view/pull/24.diff","patch_url":"https://github.com/AmpersandJS/ampersand-select-view/pull/24.patch"},"body":"...ies to submit the form"},"comment":{"url":"https://api.github.com/repos/AmpersandJS/ampersand-select-view/issues/comments/92005259","html_url":"https://github.com/AmpersandJS/ampersand-select-view/pull/24#issuecomment-92005259","issue_url":"https://api.github.com/repos/AmpersandJS/ampersand-select-view/issues/24","id":92005259,"user":{"login":"cdaringe","id":1003261,"avatar_url":"https://avatars.githubusercontent.com/u/1003261?v=3","gravatar_id":"","url":"https://api.github.com/users/cdaringe","html_url":"https://github.com/cdaringe","followers_url":"https://api.github.com/users/cdaringe/followers","following_url":"https://api.github.com/users/cdaringe/following{/other_user}","gists_url":"https://api.github.com/users/cdaringe/gists{/gist_id}","starred_url":"https://api.github.com/users/cdaringe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cdaringe/subscriptions","organizations_url":"https://api.github.com/users/cdaringe/orgs","repos_url":"https://api.github.com/users/cdaringe/repos","events_url":"https://api.github.com/users/cdaringe/events{/privacy}","received_events_url":"https://api.github.com/users/cdaringe/received_events","type":"User","site_admin":false},"created_at":"2015-04-12T07:43:41Z","updated_at":"2015-04-12T07:43:41Z","body":"hi @erd0s, apologies for the huge dialog on this one!\r\n\r\n1. the extra validation & functions that you added here was already added by @nickryall in subsequent commits (minus handleBlur)\r\n1. in our other fieldView READMEs (ref input view), we don't blur explicitly\r\n\r\n> Will not show error messages pre-submit, or if it's never had a valid value. This lets people tab-through a form without triggering a bunch of error messages.\r\n\r\nI hold the counter-opinion to @lukekarrys--I think consistency is important for the user experience across controls, specifically because of the tabbing.\r\n\r\nin light of PR #43 4.0.0 RC (which has high precedence given issue #42), I'd like to put this review on hold, or perhaps close it temporarily until the the above is merged and conflicts resolved.  Let me know your thoughts.  thanks!"}}