{"action":"created","issue":{"url":"https://api.github.com/repos/w3c/fxtf-drafts/issues/42","repository_url":"https://api.github.com/repos/w3c/fxtf-drafts","labels_url":"https://api.github.com/repos/w3c/fxtf-drafts/issues/42/labels{/name}","comments_url":"https://api.github.com/repos/w3c/fxtf-drafts/issues/42/comments","events_url":"https://api.github.com/repos/w3c/fxtf-drafts/issues/42/events","html_url":"https://github.com/w3c/fxtf-drafts/issues/42","id":175381995,"number":42,"title":"[css-motion] Grammar corrections","user":{"login":"shans","id":329053,"avatar_url":"https://avatars.githubusercontent.com/u/329053?v=3","gravatar_id":"","url":"https://api.github.com/users/shans","html_url":"https://github.com/shans","followers_url":"https://api.github.com/users/shans/followers","following_url":"https://api.github.com/users/shans/following{/other_user}","gists_url":"https://api.github.com/users/shans/gists{/gist_id}","starred_url":"https://api.github.com/users/shans/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shans/subscriptions","organizations_url":"https://api.github.com/users/shans/orgs","repos_url":"https://api.github.com/users/shans/repos","events_url":"https://api.github.com/users/shans/events{/privacy}","received_events_url":"https://api.github.com/users/shans/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/w3c/fxtf-drafts/labels/motion-1","name":"motion-1","color":"c5def5"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-09-07T00:10:43Z","updated_at":"2016-09-08T04:28:08Z","closed_at":null,"body":"(with @tabatkins)\r\n\r\n```\r\noffset-path: <url> | <angle> && <size>? && contain? | \r\n             <basic-shape> && <geometry-box>? | <path()> | none\r\n```\r\n\r\ni.e., you can have one of the following:\r\n* a url\r\n* an angle, optionally with a size and optionally with the contain keyword, in any order\r\n* a basic shape, optionally with a geometry box, in any order\r\n* a path\r\n* none \r\n\r\n```\r\noffset-rotation: [ auto | reverse ] || <angle>\r\n```\r\n\r\ni.e. you can have auto/reverse, OR an angle, OR both, in any order\r\n\r\noffset: There are multiple problems here. In addition to the problem that Eric identified (offset-position and offset-anchor are both positions)\r\n* offset-distance is ambiguous with a position when it appears next to it (so offset-distance could not appear after offset-position or offset-anchor); and\r\n* multiple properties have the value auto.\r\n\r\nBasically, it's very unlikely that this could ever work. We propose dropping the shorthand back to the original proposal (offset-path, offset-distance, offset-rotation). These are unambiguous with respect to each other, and likely to be the most commonly used parts of the specification. In other words:\r\n\r\n```\r\noffset: <offset-path> || <offset-distance> || <offset-rotation>\r\n```\r\n\r\n(Note double bar instead of double ampersand - this allows one or more of these properties to be specified whereas double ampersand requires all of them.)\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/w3c/fxtf-drafts/issues/comments/245490431","html_url":"https://github.com/w3c/fxtf-drafts/issues/42#issuecomment-245490431","issue_url":"https://api.github.com/repos/w3c/fxtf-drafts/issues/42","id":245490431,"user":{"login":"ericwilligers","id":3839073,"avatar_url":"https://avatars.githubusercontent.com/u/3839073?v=3","gravatar_id":"","url":"https://api.github.com/users/ericwilligers","html_url":"https://github.com/ericwilligers","followers_url":"https://api.github.com/users/ericwilligers/followers","following_url":"https://api.github.com/users/ericwilligers/following{/other_user}","gists_url":"https://api.github.com/users/ericwilligers/gists{/gist_id}","starred_url":"https://api.github.com/users/ericwilligers/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ericwilligers/subscriptions","organizations_url":"https://api.github.com/users/ericwilligers/orgs","repos_url":"https://api.github.com/users/ericwilligers/repos","events_url":"https://api.github.com/users/ericwilligers/events{/privacy}","received_events_url":"https://api.github.com/users/ericwilligers/received_events","type":"User","site_admin":false},"created_at":"2016-09-08T04:28:08Z","updated_at":"2016-09-08T04:28:08Z","body":"The following worked well when offset-path couldn't be an angle:\r\n `offset: <offset-path> || <offset-distance> || <offset-rotation>`\r\n\r\nBut now offset-path and offset-rotation can both be angles, so the grammar correction doesn't solve the last of the following: \r\n- offset-distance is ambiguous with a position \r\n- multiple properties have the value auto\r\n- multiple properties have position\r\n- consecutive positions are ambiguous\r\n- multiple properties have angle\r\n\r\nOne way to repair this would be to make the size non-optional in angle paths. Another would be to require property ordering in the shorthand.\r\n"}}
{"actor":{"display_login":"ericwilligers"}}