{"action":"created","issue":{"url":"https://api.github.com/repos/cfpb/regulations-site/issues/721","labels_url":"https://api.github.com/repos/cfpb/regulations-site/issues/721/labels{/name}","comments_url":"https://api.github.com/repos/cfpb/regulations-site/issues/721/comments","events_url":"https://api.github.com/repos/cfpb/regulations-site/issues/721/events","html_url":"https://github.com/cfpb/regulations-site/pull/721","id":84088166,"number":721,"title":"Performance and accessibility tweaks","user":{"login":"ascott1","id":212533,"avatar_url":"https://avatars.githubusercontent.com/u/212533?v=3","gravatar_id":"","url":"https://api.github.com/users/ascott1","html_url":"https://github.com/ascott1","followers_url":"https://api.github.com/users/ascott1/followers","following_url":"https://api.github.com/users/ascott1/following{/other_user}","gists_url":"https://api.github.com/users/ascott1/gists{/gist_id}","starred_url":"https://api.github.com/users/ascott1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ascott1/subscriptions","organizations_url":"https://api.github.com/users/ascott1/orgs","repos_url":"https://api.github.com/users/ascott1/repos","events_url":"https://api.github.com/users/ascott1/events{/privacy}","received_events_url":"https://api.github.com/users/ascott1/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-02T16:08:20Z","updated_at":"2015-06-02T20:30:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cfpb/regulations-site/pulls/721","html_url":"https://github.com/cfpb/regulations-site/pull/721","diff_url":"https://github.com/cfpb/regulations-site/pull/721.diff","patch_url":"https://github.com/cfpb/regulations-site/pull/721.patch"},"body":"This does the following:\r\n\r\n- Goes back to loading local font files instead of a font provider CDN (sorry @KimberlyMunoz :disappointed:)\r\n- Adds appropriate ARIA roles\r\n- Loads Modernizr inline in the `<head>` to remove the request (this is ugly but seems worthwhile since we need it on the initial page load)\r\n\r\n## Notes\r\n\r\n- I wanted to add `async` to our `script` tags but Google Tag Manager's reliance on jQuery meant that this broke analytics. I'm hoping to work with @mistergone to fix this as using `async` made page loads feel snappy\r\n- I also tested using Filament Group's [loadCSS](https://github.com/filamentgroup/loadCSS), but the flash of unstyled content and some jankiness with scrolling led me to abandoning it for now\r\n\r\n## Review\r\n\r\n@KimberlyMunoz "},"comment":{"url":"https://api.github.com/repos/cfpb/regulations-site/issues/comments/108089722","html_url":"https://github.com/cfpb/regulations-site/pull/721#issuecomment-108089722","issue_url":"https://api.github.com/repos/cfpb/regulations-site/issues/721","id":108089722,"user":{"login":"KimberlyMunoz","id":1860176,"avatar_url":"https://avatars.githubusercontent.com/u/1860176?v=3","gravatar_id":"","url":"https://api.github.com/users/KimberlyMunoz","html_url":"https://github.com/KimberlyMunoz","followers_url":"https://api.github.com/users/KimberlyMunoz/followers","following_url":"https://api.github.com/users/KimberlyMunoz/following{/other_user}","gists_url":"https://api.github.com/users/KimberlyMunoz/gists{/gist_id}","starred_url":"https://api.github.com/users/KimberlyMunoz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/KimberlyMunoz/subscriptions","organizations_url":"https://api.github.com/users/KimberlyMunoz/orgs","repos_url":"https://api.github.com/users/KimberlyMunoz/repos","events_url":"https://api.github.com/users/KimberlyMunoz/events{/privacy}","received_events_url":"https://api.github.com/users/KimberlyMunoz/received_events","type":"User","site_admin":false},"created_at":"2015-06-02T20:30:42Z","updated_at":"2015-06-02T20:30:42Z","body":":fallen_leaf: \r\n\r\nReviewing this and your other pull request between meetings today."}}