{"action":"created","issue":{"url":"https://api.github.com/repos/Polymer/polymer/issues/3240","labels_url":"https://api.github.com/repos/Polymer/polymer/issues/3240/labels{/name}","comments_url":"https://api.github.com/repos/Polymer/polymer/issues/3240/comments","events_url":"https://api.github.com/repos/Polymer/polymer/issues/3240/events","html_url":"https://github.com/Polymer/polymer/pull/3240","id":124076490,"number":3240,"title":"Make tests succeed in Safari","user":{"login":"rubenstolk","id":692644,"avatar_url":"https://avatars.githubusercontent.com/u/692644?v=3","gravatar_id":"","url":"https://api.github.com/users/rubenstolk","html_url":"https://github.com/rubenstolk","followers_url":"https://api.github.com/users/rubenstolk/followers","following_url":"https://api.github.com/users/rubenstolk/following{/other_user}","gists_url":"https://api.github.com/users/rubenstolk/gists{/gist_id}","starred_url":"https://api.github.com/users/rubenstolk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rubenstolk/subscriptions","organizations_url":"https://api.github.com/users/rubenstolk/orgs","repos_url":"https://api.github.com/users/rubenstolk/repos","events_url":"https://api.github.com/users/rubenstolk/events{/privacy}","received_events_url":"https://api.github.com/users/rubenstolk/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Polymer/polymer/labels/cla:%20yes","name":"cla: yes","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-28T14:48:14Z","updated_at":"2015-12-28T15:51:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Polymer/polymer/pulls/3240","html_url":"https://github.com/Polymer/polymer/pull/3240","diff_url":"https://github.com/Polymer/polymer/pull/3240.diff","patch_url":"https://github.com/Polymer/polymer/pull/3240.patch"},"body":"This PR solves issue https://github.com/Polymer/polymer/issues/3166\r\n\r\nAs far as I can see, the element in the test suite is given `bottom`, `left` and `right` properties, however the element doesn't have any value for `position` (default `static`). The difference between Chrome and Safari for `window.getComputedStyle(document.querySelector('child-of-child-with-var')).bottom` is:\r\n\r\n- Safari: `\"auto\"`\r\n- Chrome: `\"9px\"`\r\n\r\nApplying `position: relative` solves this."},"comment":{"url":"https://api.github.com/repos/Polymer/polymer/issues/comments/167591986","html_url":"https://github.com/Polymer/polymer/pull/3240#issuecomment-167591986","issue_url":"https://api.github.com/repos/Polymer/polymer/issues/3240","id":167591986,"user":{"login":"TimvdLippe","id":5948271,"avatar_url":"https://avatars.githubusercontent.com/u/5948271?v=3","gravatar_id":"","url":"https://api.github.com/users/TimvdLippe","html_url":"https://github.com/TimvdLippe","followers_url":"https://api.github.com/users/TimvdLippe/followers","following_url":"https://api.github.com/users/TimvdLippe/following{/other_user}","gists_url":"https://api.github.com/users/TimvdLippe/gists{/gist_id}","starred_url":"https://api.github.com/users/TimvdLippe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TimvdLippe/subscriptions","organizations_url":"https://api.github.com/users/TimvdLippe/orgs","repos_url":"https://api.github.com/users/TimvdLippe/repos","events_url":"https://api.github.com/users/TimvdLippe/events{/privacy}","received_events_url":"https://api.github.com/users/TimvdLippe/received_events","type":"User","site_admin":false},"created_at":"2015-12-28T15:51:17Z","updated_at":"2015-12-28T15:51:17Z","body":"The build errored on FireFox due to a Selenium error of not being able to connect with 45000 ms. I am not sure if that is related to this issue.\r\n\r\nHowever, if the CSS parsing of Safari and the other browsers are equal for `position`, I think the issue can be solved by:\r\nLine 238\r\n```css\r\n--variable-into-first-variable: relative;\r\n```\r\nLine 274\r\n```css\r\nposition: var(--variable-from-other-variable);\r\n```\r\n\r\nThe `position: relative;` on line 251 can be removed again.\r\n\r\nAnother thing is that the original PR was reverted. I am not sure if this change would be applied, if the merge to master will contain the original fixes, or just these 2 line-changes. CC: @azakus "}}