{"action":"created","issue":{"url":"https://api.github.com/repos/ampproject/amphtml/issues/3578","repository_url":"https://api.github.com/repos/ampproject/amphtml","labels_url":"https://api.github.com/repos/ampproject/amphtml/issues/3578/labels{/name}","comments_url":"https://api.github.com/repos/ampproject/amphtml/issues/3578/comments","events_url":"https://api.github.com/repos/ampproject/amphtml/issues/3578/events","html_url":"https://github.com/ampproject/amphtml/issues/3578","id":160264480,"number":3578,"title":"AMP validator forbids \"style\" attribute in <stop> svg element. ","user":{"login":"nbeloglazov","id":252053,"avatar_url":"https://avatars.githubusercontent.com/u/252053?v=3","gravatar_id":"","url":"https://api.github.com/users/nbeloglazov","html_url":"https://github.com/nbeloglazov","followers_url":"https://api.github.com/users/nbeloglazov/followers","following_url":"https://api.github.com/users/nbeloglazov/following{/other_user}","gists_url":"https://api.github.com/users/nbeloglazov/gists{/gist_id}","starred_url":"https://api.github.com/users/nbeloglazov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nbeloglazov/subscriptions","organizations_url":"https://api.github.com/users/nbeloglazov/orgs","repos_url":"https://api.github.com/users/nbeloglazov/repos","events_url":"https://api.github.com/users/nbeloglazov/events{/privacy}","received_events_url":"https://api.github.com/users/nbeloglazov/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ampproject/amphtml/labels/Related%20to:%20Validator","name":"Related to: Validator","color":"d4c5f9"}],"state":"open","locked":false,"assignee":{"login":"Gregable","id":25519,"avatar_url":"https://avatars.githubusercontent.com/u/25519?v=3","gravatar_id":"","url":"https://api.github.com/users/Gregable","html_url":"https://github.com/Gregable","followers_url":"https://api.github.com/users/Gregable/followers","following_url":"https://api.github.com/users/Gregable/following{/other_user}","gists_url":"https://api.github.com/users/Gregable/gists{/gist_id}","starred_url":"https://api.github.com/users/Gregable/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Gregable/subscriptions","organizations_url":"https://api.github.com/users/Gregable/orgs","repos_url":"https://api.github.com/users/Gregable/repos","events_url":"https://api.github.com/users/Gregable/events{/privacy}","received_events_url":"https://api.github.com/users/Gregable/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/ampproject/amphtml/milestones/18","html_url":"https://github.com/ampproject/amphtml/milestones/Current","labels_url":"https://api.github.com/repos/ampproject/amphtml/milestones/18/labels","id":1739456,"number":18,"title":"Current","description":"This milestone tracks issues currently being worked on.","creator":{"login":"ericlindley-g","id":16807766,"avatar_url":"https://avatars.githubusercontent.com/u/16807766?v=3","gravatar_id":"","url":"https://api.github.com/users/ericlindley-g","html_url":"https://github.com/ericlindley-g","followers_url":"https://api.github.com/users/ericlindley-g/followers","following_url":"https://api.github.com/users/ericlindley-g/following{/other_user}","gists_url":"https://api.github.com/users/ericlindley-g/gists{/gist_id}","starred_url":"https://api.github.com/users/ericlindley-g/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ericlindley-g/subscriptions","organizations_url":"https://api.github.com/users/ericlindley-g/orgs","repos_url":"https://api.github.com/users/ericlindley-g/repos","events_url":"https://api.github.com/users/ericlindley-g/events{/privacy}","received_events_url":"https://api.github.com/users/ericlindley-g/received_events","type":"User","site_admin":false},"open_issues":40,"closed_issues":32,"state":"open","created_at":"2016-04-29T23:08:57Z","updated_at":"2016-06-17T16:54:27Z","due_on":null,"closed_at":null},"comments":12,"created_at":"2016-06-14T19:30:55Z","updated_at":"2016-06-17T17:00:19Z","closed_at":null,"body":"Please walk through the following steps for reporting an issue about AMP:\r\n\r\n- Please ask general questions about AMP over on stackoverflow under the AMP HTML tag: http://stackoverflow.com/questions/tagged/amp-html\r\n- Please ask/post questions/problems related to Google Search in Google's AMP forum: https://goo.gl/utQ1KZ\r\n\r\n## Short description of your issue:\r\nAMP validator disallows \"style\" attribute in <stop> svg element. \r\n\r\nError:\r\n`The attribute 'style' may not appear in tag 'radialgradient > stop'. (see https://www.ampproject.org/docs/reference/spec.html#svg)`\r\n\r\n## How do we reproduce the issue?\r\n\r\nhttp://jsbin.com/karukoyoqi/edit?html,output\r\nOpen output in new window, add `#development=1` and refresh. \r\n\r\nAccording to https://www.ampproject.org/docs/reference/spec.html#svg 'style' attribute is allowed.\r\n\r\n## What browsers are affected?\r\n\r\nValidator issue. Probably all browsers? \r\n\r\n## Which AMP version is affected?\r\n\r\nVersion 1464987951323\r\n"},"comment":{"url":"https://api.github.com/repos/ampproject/amphtml/issues/comments/226824128","html_url":"https://github.com/ampproject/amphtml/issues/3578#issuecomment-226824128","issue_url":"https://api.github.com/repos/ampproject/amphtml/issues/3578","id":226824128,"user":{"login":"cramforce","id":89679,"avatar_url":"https://avatars.githubusercontent.com/u/89679?v=3","gravatar_id":"","url":"https://api.github.com/users/cramforce","html_url":"https://github.com/cramforce","followers_url":"https://api.github.com/users/cramforce/followers","following_url":"https://api.github.com/users/cramforce/following{/other_user}","gists_url":"https://api.github.com/users/cramforce/gists{/gist_id}","starred_url":"https://api.github.com/users/cramforce/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cramforce/subscriptions","organizations_url":"https://api.github.com/users/cramforce/orgs","repos_url":"https://api.github.com/users/cramforce/repos","events_url":"https://api.github.com/users/cramforce/events{/privacy}","received_events_url":"https://api.github.com/users/cramforce/received_events","type":"User","site_admin":false},"created_at":"2016-06-17T17:00:19Z","updated_at":"2016-06-17T17:00:19Z","body":"@nbeloglazov Reasons against style attribute\r\n\r\n- Feasibility of using a CSP for styles without nonces (which can't be used in publicly cacheable docs)\r\n- Can make doc very slow and bloated if overused\r\n- No, good idea how to enforce limits of usage\r\n\r\nReasons pro:\r\n\r\n- very convenient sometimes\r\n- heavily used by SVG generation tools\r\n- better semantic match for content dependent styles"}}
{"actor":{"display_login":"cramforce"}}