From 99c6cfdba88e0c30e53a3077d0cb0359be2822d5 Mon Sep 17 00:00:00 2001 From: Narasimha-sc <166327228+Narasimha-sc@users.noreply.github.com> Date: Wed, 12 Aug 2026 14:30:09 +0000 Subject: [PATCH] docs: drop file path references from animated images plan --- plans/2026-08-11-desktop-animated-images.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plans/2026-08-11-desktop-animated-images.md b/plans/2026-08-11-desktop-animated-images.md index 9b4b18e8d6..7d3a3ad612 100644 --- a/plans/2026-08-11-desktop-animated-images.md +++ b/plans/2026-08-11-desktop-animated-images.md @@ -4,14 +4,14 @@ `SimpleAndAnimatedImageView` on desktop drew a single `BitmapPainter` and carried the marker `// LALAL make it animated too`. Android decodes animations with coil, iOS with SwiftyGif, and desktop showed -the first frame and stopped. `ImageFullScreenView.kt` carried a matching marker over the image branch. +the first frame and stopped. `ImageFullScreenView` carried a matching marker over the image branch. ## Why this shape **Skia's `Codec`, which skiko already puts on the desktop classpath.** No new dependency. It decodes both GIF and animated WebP, reports per-frame durations and repeat counts, and supports random access into frames. -**Not `components-animatedimage`** (already declared in `common/build.gradle.kts`, unused). Its `animate()` +**Not `components-animatedimage`** (already a declared dependency, unused). Its `animate()` ignores the result of `allocPixels` and decodes inside composition. A 35-byte GIF declaring 65535x65535 asks for a 17GB raster; `allocPixels` returns false, and the following `readPixels` throws `IllegalArgumentException` from inside the composition — a remote crash from anyone who can send a file. It