-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Firefox + amp-img + Polymer not rendering properly #12388
Comments
https://amp-by-example-staging.appspot.com/introduction/hello_world/ on Firefox is now working for me. If you look at: https://rep.repubblica.it/pwa/intervista/2017/12/08/news/laura_boldrini_leggi_e_condivisione_di_valori_contro_il_neofascismo-183530302/ is also loading the image, but if you arrive to the page via a navigation, from https://rep.repubblica.it/ click on where it says "Boldrini", then the head image does not load. @justforfun FYI |
@kul3r4 : The only difference I can see comparing the console logs of the two cases is that when you load directly the URL the following message appears: Powered by AMP ⚡ HTML shadows – Version 1512677960104 viewer-impl.js:1050:38 |
@justforfun Sorry, I was testing the wrong URL, I am still able to reproduce it on https://amp-by-example-staging.appspot.com/polymer/detail/introduction/hello_world/. This only happens when landing on the page via navigation, while the page works if you just load the URL, same thing as it happens on rep.repubblica.it. |
If it can be of any help, it seems that the problem is not occurring in an pwa based on Polymer 1. |
We haven't lost track of this, but are still awaiting some folks returning from holidays who could take a look. |
@justforfun I just tried the following:
Could you confirm this bug is still happening? |
I attempted to reproduce yesterday and today and I was unable. Are you aware of anything that changed on your end to resolve the problem? |
I'm not able to reproduce the bug anymore but on the FF browser console I see the error: The error doesn't occur on Chrome |
I can reproduce that error, but I have not yet found the root cause. It looks like an event polyfill in Polymer that runs for firefox but not chrome might be related to the problem |
I will close this issue for now. If the new error is still occurring for you please open a new issue so we can track it. |
@cvializ could you please reopen this as it is still happening |
This issue hasn't been updated in awhile. @cvializ Do you have any updates? |
/to @danielrozenberg |
This issue hasn't been updated in awhile. @danielrozenberg Do you have any updates? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
In Firefox,
amp-img
might be misbehaving either when loaded byshadow-v0.js
and/or when rendered inside a Polymer app.Example: https://amp-by-example-staging.appspot.com/polymer/detail/introduction/hello_world/
Split off from #12132
/cc @kul3r4 @aghassemi
The text was updated successfully, but these errors were encountered: