Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

The offsetParent is wrong in the attached callback #2193

Closed
notwaldorf opened this issue Aug 4, 2015 · 2 comments
Closed

The offsetParent is wrong in the attached callback #2193

notwaldorf opened this issue Aug 4, 2015 · 2 comments
Assignees
@sorvell
Labels
p1

Comments

@notwaldorf
Copy link
Contributor

on Chrome 43.0.2357.134 (Official Build) (64-bit), this code doesn't have an offsetParent. It works on FF and Safari:

https://gist.github.com/notwaldorf/44da856a3debdc8ab289
@sorvell
Copy link
Contributor

sorvell commented Aug 4, 2015

This looks like a Chrome bug to me. I've filed:

https://code.google.com/p/chromium/issues/detail?id=516550

@sorvell
Copy link
Contributor

sorvell commented Aug 4, 2015

After discussing this with some Blink engineers, it looks like fixing the bug will take a non-trivial amount of time. In the meantime, Polymer can use a slightly beefed up version of Polymer.ImportStatus.whenLoaded(cb) to defer calling attached slightly to avoid querying style/layout data too soon.

@sorvell sorvell closed this as completed in 2bffc4c Aug 5, 2015
kevinpschaaf added a commit that referenced this issue Aug 5, 2015
@kevinpschaaf
Merge pull request #2202 from Polymer/fix-2193
0892e93 
Fixes #2193: Implements workaround for https://code.google.com/p/chro…
@blasten blasten mentioned this issue Aug 6, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sorvell sorvell

Labels
p1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@sorvell @kevinpschaaf @notwaldorf