Freegle / Iznik / 5635

72%

master: 70%

Build Type

Pull #232

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 Haiku 4.5 <noreply@anthropic.com>

Pull Request Pull Request #232: test(coverage): unit tests for utils package — Flex types, Blur, TidyName, Haversine

Coverage Stats

13361 of 20296 branches covered (65.83%)

Branch coverage included in aggregate %.

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

345 existing lines in 5 files now uncovered.

96192 of 132791 relevant lines covered (72.44%)

21.46 hits per line