{"action":"created","issue":{"url":"https://api.github.com/repos/zalmoxisus/redux-devtools-extension/issues/54","repository_url":"https://api.github.com/repos/zalmoxisus/redux-devtools-extension","labels_url":"https://api.github.com/repos/zalmoxisus/redux-devtools-extension/issues/54/labels{/name}","comments_url":"https://api.github.com/repos/zalmoxisus/redux-devtools-extension/issues/54/comments","events_url":"https://api.github.com/repos/zalmoxisus/redux-devtools-extension/issues/54/events","html_url":"https://github.com/zalmoxisus/redux-devtools-extension/pull/54","id":133108457,"number":54,"title":"[WIP] Clean up the options page","user":{"login":"iamakulov","id":2953267,"avatar_url":"https://avatars.githubusercontent.com/u/2953267?v=3","gravatar_id":"","url":"https://api.github.com/users/iamakulov","html_url":"https://github.com/iamakulov","followers_url":"https://api.github.com/users/iamakulov/followers","following_url":"https://api.github.com/users/iamakulov/following{/other_user}","gists_url":"https://api.github.com/users/iamakulov/gists{/gist_id}","starred_url":"https://api.github.com/users/iamakulov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iamakulov/subscriptions","organizations_url":"https://api.github.com/users/iamakulov/orgs","repos_url":"https://api.github.com/users/iamakulov/repos","events_url":"https://api.github.com/users/iamakulov/events{/privacy}","received_events_url":"https://api.github.com/users/iamakulov/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":28,"created_at":"2016-02-11T22:48:29Z","updated_at":"2016-07-17T08:14:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/zalmoxisus/redux-devtools-extension/pulls/54","html_url":"https://github.com/zalmoxisus/redux-devtools-extension/pull/54","diff_url":"https://github.com/zalmoxisus/redux-devtools-extension/pull/54.diff","patch_url":"https://github.com/zalmoxisus/redux-devtools-extension/pull/54.patch"},"body":"I’ve cleaned up the extension options page slightly and improved accessibility. Before and after:\r\n\r\n![The options page before the cleanup and after it](https://cloud.githubusercontent.com/assets/2953267/12994238/fa63dbf6-d12f-11e5-85c0-808a1b0560c9.png)\r\n\r\n\r\n----\r\n\r\n**The list of changes:**\r\n\r\n*26b7f33 – Replace spans with labels for better accessibility*\r\nThis makes focusing fields and checking checkboxes easier. When you add a label and link it with an input, this label becomes clickable, and you can click it to check the checkbox or focus the input it’s linked with.\r\n\r\n*186f087 – Implement proper wrapping*\r\nIn forms, line breaks are usually created with `<br/>`. This commit removes the `.caption` width limitation to make wide captions look properly and implements proper line breaks instead.\r\n\r\n*9300cbd – Resize checkboxes and align them*\r\nA large checkbox is blurry, a smaller one looks way better. Also, `baseline` alignment is used instead of the default `middle` one because it also looks better.\r\n\r\n*c2bdbd2 – Swap labels and checkboxes*\r\nMake the labels appear after the checkboxes to follow common UI patterns. Also, include the captions into the labels to increase the clickable area.\r\n\r\n*861e1f7 – Make text input non-bold*\r\nThe bold text looks like a design exception and seem to have no meaning. This in fact is just a guess, so please correct me if I’m wrong.\r\n\r\nLooking forward to your comments."},"comment":{"url":"https://api.github.com/repos/zalmoxisus/redux-devtools-extension/issues/comments/233170804","html_url":"https://github.com/zalmoxisus/redux-devtools-extension/pull/54#issuecomment-233170804","issue_url":"https://api.github.com/repos/zalmoxisus/redux-devtools-extension/issues/54","id":233170804,"user":{"login":"iamakulov","id":2953267,"avatar_url":"https://avatars.githubusercontent.com/u/2953267?v=3","gravatar_id":"","url":"https://api.github.com/users/iamakulov","html_url":"https://github.com/iamakulov","followers_url":"https://api.github.com/users/iamakulov/followers","following_url":"https://api.github.com/users/iamakulov/following{/other_user}","gists_url":"https://api.github.com/users/iamakulov/gists{/gist_id}","starred_url":"https://api.github.com/users/iamakulov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iamakulov/subscriptions","organizations_url":"https://api.github.com/users/iamakulov/orgs","repos_url":"https://api.github.com/users/iamakulov/repos","events_url":"https://api.github.com/users/iamakulov/events{/privacy}","received_events_url":"https://api.github.com/users/iamakulov/received_events","type":"User","site_admin":false},"created_at":"2016-07-17T08:14:04Z","updated_at":"2016-07-17T08:14:04Z","body":"@zalmoxisus, I think I’m done here. Is there something we should yet do before merging?\r\n\r\nThe Travis build is failed for some reason, however, both AppVeyor and my local `npm run test` are OK."}}
{"actor":{"display_login":"iamakulov"}}