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.
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
Polymer unresolved to prevent FOUC doesn't work when a second attribute exists #20826
Comments
mmm - I can totally see it if I open your site, but I haven't yet got to do a local repro. If I refresh the site, I don't see it though. Do you see this also before compiling the app? Only in JS or in Dartium too? I'm not sure if this is in our side or comes all the way from the polymer.js side of things. Added Pkg-Polymer, Area-Pkg, Triaged labels. |
Aha, we just figured out what is the issue. Turns out that the second attribute is not really causing this. The problem was that our build is moving the platform.js script tag to the <body> so initially <body unresolved> is there, but the style to make unresolved invisible is not there yet. Platform.js appends a style tag to HEAD to make that work. That logic relies on the fact that platform.js was loaded in <HEAD>. We need to fix this in our compiler. The best workaround for now is to edit the generated code and move platform.js and dart_support.js to the <HEAD> of the generated .html. Removed Priority-Unassigned label. |
Set owner to @jakemac53. |
Going with the slightly more general approach of not moving anything into the body until we find the first import. |
Fixed in r40511 Added Fixed label. |
This comment was originally written by davenot...@gmail.com Rad! Doing a "vi build/web/index.html" to move dart_support.js into head every time I build for production is no fun. :) I'll have to wait for it to hit stable, though. |
This issue has been moved to dart-archive/polymer-dart#289. |
This issue was originally filed by dave...@gmail.com
What steps will reproduce the problem?
What version of the product are you using?
SDK 1.6, Polymer 0.13.
Please provide any additional information below.
The text was updated successfully, but these errors were encountered: