Setting media field to loading="eager" doesn’t work when using the media_thumbnail field formatter

Patch MR !2873 works for me on Drupal 10.0.4.

Still NW for addition of tests and the MR feedback.

This adds tests and responds to the feedback on the MR. Since I didn't have access on the original MR to change its base, I've opened a new on that is set to merge into 10.1.x.

Verified that the MR in #14 resolves the issue. I tested both eager and lazy loading, and the patch works as expected.

I'm attaching the test as a patch to ensure that it fails properly. Assuming that the patch containing the tests fail, this is RTBC.

I also hope we can backport this to 9.5.x, as this is a but that'll likely affect a number of sites.

The last submitted patch, 16: 3307409-16-TEST-ONLY-FAIL.patch, failed testing. View results →

Test only patch failed! RTBC!

The last submitted patch, 16: 3307409-16-TEST-ONLY-FAIL.patch, failed testing. View results →

Committed/pushed to 10.1.x, thanks!

yay!

@catch: would it be appropriate to also backport this to 9.5.x and 10.0.x?

This is something that I've run into several times in my projects :D

We could but it would be nice to get 10.0 and 9.5 test runs.

Makes sense!

Patches are attached. Note this is the exact same patch as the commit in #21

We could but it would be nice to get 10.0 and 9.5 test runs.

Test runs completed. Setting to RTBC so it doesn't slip through the cracks. Thanks!

Backported to both branches, thanks!

Thank you!

Automatically closed - issue fixed for 2 weeks with no activity.