Problem/Motivation

As discussed in #3238501-35: Remove empty and deprecated IE11 polyfill library stub entries in 11.0.0:

If everything supports
now, can we remove the fallback image in a followup?

and #3238501-36: Remove empty and deprecated IE11 polyfill library stub entries in 11.0.0:

I would say yes.

Let's remove the fallback image functionality.

Postponed on #3238501: Remove empty and deprecated IE11 polyfill library stub entries in 11.0.0

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Comments

Spokje created an issue. See original summary.

catch’s picture

Looked up https://developer.mozilla.org/en-US/docs/Web/HTML/Element/picture

The fallback is used in the case that the browser doesn't support picture, but it's also used in the case that none of the source elements matched. I guess if you had a breakpoint for <200px, and a responsive image that only used that breakpoint, and then you viewed at 1900px. So we might need to keep it for that case anyway?

longwave’s picture

The img tag contains the alt attribute as well, so we probably still need to keep it. I suppose modern browsers will generally do the right thing and not download two images when they only intend to display one (unless the window size changes, etc).

Maybe we just need to update the comment that talks about picture not being supported.

Version: 10.0.x-dev » 11.x-dev

Drupal core is moving towards using a “main” branch. As an interim step, a new 11.x branch has been opened, as Drupal.org infrastructure cannot currently fully support a branch named main. New developments and disruptive changes should now be targeted for the 11.x branch. For more information, see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.