{"action":"created","issue":{"url":"https://api.github.com/repos/Financial-Times/n-image/issues/35","repository_url":"https://api.github.com/repos/Financial-Times/n-image","labels_url":"https://api.github.com/repos/Financial-Times/n-image/issues/35/labels{/name}","comments_url":"https://api.github.com/repos/Financial-Times/n-image/issues/35/comments","events_url":"https://api.github.com/repos/Financial-Times/n-image/issues/35/events","html_url":"https://github.com/Financial-Times/n-image/pull/35","id":167023836,"number":35,"title":"Adds a empty alt attribute when no alt text is set","user":{"login":"GlynnPhillips","id":524573,"avatar_url":"https://avatars.githubusercontent.com/u/524573?v=3","gravatar_id":"","url":"https://api.github.com/users/GlynnPhillips","html_url":"https://github.com/GlynnPhillips","followers_url":"https://api.github.com/users/GlynnPhillips/followers","following_url":"https://api.github.com/users/GlynnPhillips/following{/other_user}","gists_url":"https://api.github.com/users/GlynnPhillips/gists{/gist_id}","starred_url":"https://api.github.com/users/GlynnPhillips/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GlynnPhillips/subscriptions","organizations_url":"https://api.github.com/users/GlynnPhillips/orgs","repos_url":"https://api.github.com/users/GlynnPhillips/repos","events_url":"https://api.github.com/users/GlynnPhillips/events{/privacy}","received_events_url":"https://api.github.com/users/GlynnPhillips/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-07-22T11:14:37Z","updated_at":"2016-07-22T14:18:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Financial-Times/n-image/pulls/35","html_url":"https://github.com/Financial-Times/n-image/pull/35","diff_url":"https://github.com/Financial-Times/n-image/pull/35.diff","patch_url":"https://github.com/Financial-Times/n-image/pull/35.patch"},"body":"Allows for either no alt attr or `alt=\"\"`\r\nAlso adds `role=\"presentation\"` when the alt is empty"},"comment":{"url":"https://api.github.com/repos/Financial-Times/n-image/issues/comments/234556016","html_url":"https://github.com/Financial-Times/n-image/pull/35#issuecomment-234556016","issue_url":"https://api.github.com/repos/Financial-Times/n-image/issues/35","id":234556016,"user":{"login":"debugwand","id":996106,"avatar_url":"https://avatars.githubusercontent.com/u/996106?v=3","gravatar_id":"","url":"https://api.github.com/users/debugwand","html_url":"https://github.com/debugwand","followers_url":"https://api.github.com/users/debugwand/followers","following_url":"https://api.github.com/users/debugwand/following{/other_user}","gists_url":"https://api.github.com/users/debugwand/gists{/gist_id}","starred_url":"https://api.github.com/users/debugwand/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/debugwand/subscriptions","organizations_url":"https://api.github.com/users/debugwand/orgs","repos_url":"https://api.github.com/users/debugwand/repos","events_url":"https://api.github.com/users/debugwand/events{/privacy}","received_events_url":"https://api.github.com/users/debugwand/received_events","type":"User","site_admin":false},"created_at":"2016-07-22T14:18:07Z","updated_at":"2016-07-22T14:18:07Z","body":"looks good to me.\r\n\r\nis there somewhere to document a warning that images inside links must have alt text? In some situations, screenreaders will read out the image src if there is empty alt text. I don't think you can actually test for that inside n-image module..."}}
{"actor":{"display_login":"debugwand"}}