-
Notifications
You must be signed in to change notification settings - Fork 6
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
Providing a fallback image inside <x-picture> results in fallback always downloading #1
Comments
can we wrap it in noscript to make it inert? |
We could (and should - or shall), but that only addresses SEO -- which, in theory is already addressed by the It doesn't address a failure in x-picture or polymer2. But more important to my over-excited sense of principle, it means it won't mirror the proposal exactly (<x-picture> vs <picture> withstanding)3 In the end, I don't have a solution I am excited about, but <noscript> at least begins to address some issues, and it's far better than downloading an extra image that will never be seen. I wonder if it's better than not including an image fallback at all? 1Numbers on JS disabled are incredibly hard to come by, with the afore citation from 2010 the most recent -- nearly irrelevant at this point. |
The guts of the implementation (in the shadow DOM) should not matter too much, IMHO. We gotta use whatever tools we have at our disposal to make it work. |
@marcoscaceres I'm second guessing how I understood your original comment, which makes me think I might have been too vague in the issue's description. The current problem is the fallback in the markup. The implementation in the shadow DOM is solid as far as when things download. If we include |
Sorry, I've been sitting on this for a while because of the src-n proposal. It will likely have significant implications on what we do here (specially if src-n kills the picture element). On Monday, September 16, 2013 at 11:22 PM, Cory wrote:
|
Yes. I've been watching that closely. I've been meaning to find some time to prollyfill that as well to understand how well it would work.
|
Yeah, that was my plan also :) I've made a start here (still using the old syntax): Will try to update it next week. Happy to collaborate. |
Not sure there is a solution to this. It illustrates one of several things that a non-native solution can't do or can't do well.
The text was updated successfully, but these errors were encountered: