{"action":"created","issue":{"url":"https://api.github.com/repos/Financial-Times/n-header-footer/issues/39","labels_url":"https://api.github.com/repos/Financial-Times/n-header-footer/issues/39/labels{/name}","comments_url":"https://api.github.com/repos/Financial-Times/n-header-footer/issues/39/comments","events_url":"https://api.github.com/repos/Financial-Times/n-header-footer/issues/39/events","html_url":"https://github.com/Financial-Times/n-header-footer/issues/39","id":102753106,"number":39,"title":"Logo should not be an H1","user":{"login":"i-like-robots","id":271645,"avatar_url":"https://avatars.githubusercontent.com/u/271645?v=3","gravatar_id":"","url":"https://api.github.com/users/i-like-robots","html_url":"https://github.com/i-like-robots","followers_url":"https://api.github.com/users/i-like-robots/followers","following_url":"https://api.github.com/users/i-like-robots/following{/other_user}","gists_url":"https://api.github.com/users/i-like-robots/gists{/gist_id}","starred_url":"https://api.github.com/users/i-like-robots/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/i-like-robots/subscriptions","organizations_url":"https://api.github.com/users/i-like-robots/orgs","repos_url":"https://api.github.com/users/i-like-robots/repos","events_url":"https://api.github.com/users/i-like-robots/events{/privacy}","received_events_url":"https://api.github.com/users/i-like-robots/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-08-24T09:51:50Z","updated_at":"2015-08-24T12:59:32Z","closed_at":null,"body":"Logos should be an image and not marked up as a heading. Keep the top level heading for the main page content (no browser yet implements the specced document outline w/ multiple headers that I know of). The document title is fine for naming the source of the document and orientating a user.\r\n\r\n```html\r\n<a href=\"...\" title=\"Go to homepage\">\r\n  <img src=\"...\" srcset=\"...\" rel=\"logo\" alt=\"Financial Times\">\r\n</a>\r\n```\r\n\r\nhttps://github.com/Financial-Times/n-header-footer/blob/master/templates/header.html#L17-L21\r\n\r\nSome more reasoning:\r\n- http://csswizardry.com/2010/10/your-logo-is-an-image-not-a-h1/\r\n- http://www.amberweinberg.com/the-logo-to-h1-or-not-to-h1/\r\n\r\nCC @charypar "},"comment":{"url":"https://api.github.com/repos/Financial-Times/n-header-footer/issues/comments/134188847","html_url":"https://github.com/Financial-Times/n-header-footer/issues/39#issuecomment-134188847","issue_url":"https://api.github.com/repos/Financial-Times/n-header-footer/issues/39","id":134188847,"user":{"login":"michaelandrew","id":1167544,"avatar_url":"https://avatars.githubusercontent.com/u/1167544?v=3","gravatar_id":"","url":"https://api.github.com/users/michaelandrew","html_url":"https://github.com/michaelandrew","followers_url":"https://api.github.com/users/michaelandrew/followers","following_url":"https://api.github.com/users/michaelandrew/following{/other_user}","gists_url":"https://api.github.com/users/michaelandrew/gists{/gist_id}","starred_url":"https://api.github.com/users/michaelandrew/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/michaelandrew/subscriptions","organizations_url":"https://api.github.com/users/michaelandrew/orgs","repos_url":"https://api.github.com/users/michaelandrew/repos","events_url":"https://api.github.com/users/michaelandrew/events{/privacy}","received_events_url":"https://api.github.com/users/michaelandrew/received_events","type":"User","site_admin":false},"created_at":"2015-08-24T12:59:32Z","updated_at":"2015-08-24T12:59:32Z","body":"@i-like-robots 100%. We are already in talks to completely get rid of sprites and any sort of icon png as it is with a look towards vectoring, however `srcset` and `sizes` may not be an option for us at the moment without polyfills as we have to support IE9."}}