Freegle / Iznik / 5619

72%

master: 70%

Build Type

Pull #239

circleci

Committed by edwh

Commit Message

fix(nuxt3): add loading=lazy to NuxtPicture for externaluid images in MessageSummary

Images with externaluid (Uploadcare) were loaded eagerly because the NuxtPicture
component had no `loading` attribute, unlike OurUploadedImage and ProxyImage which
both correctly default to loading="lazy". On slow mobile connections (e.g. Honor 200
lite on 4G) this caused all 10 visible post images to download simultaneously,
saturating bandwidth and freezing the page for 30+ seconds.

Fix: add `:loading="preload ? 'eager' : 'lazy'"` to match the existing pattern used
by OurUploadedImage and ProxyImage. First two cards (preload=true) still load eagerly
for above-fold LCP; remaining cards are lazy as intended.

Tests: two new assertions in MessageSummary.spec.js and MessageSummaryLazyLoading.spec.js
confirm the :loading attribute is present in the externaluid NuxtPicture block.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Pull Request Pull Request #239: fix(nuxt3): add loading=lazy to NuxtPicture for externaluid images

Coverage Stats

13375 of 20279 branches covered (65.95%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

357 existing lines in 5 files now uncovered.

85370 of 116572 relevant lines covered (73.23%)

22.28 hits per line