WIP: fixes

Author: sergey3bv

crates/notedeck/src/imgcache.rs

@@ -192,21 +192,6 @@ impl MediaCache {
         Ok(())
     }
 
-    /// Writes WebP images (static or animated) to the cache
-    ///
-    /// This function handles both static (single-frame) and animated (multi-frame)
-    /// WebP images using the webp crate. For static images, it uses lossless encoding.
-    /// For animated images, it encodes all frames with their respective delays.
-    ///
-    /// # Arguments
-    ///
-    /// * `cache_dir` - The cache directory path
-    /// * `url` - The URL of the image (used for cache key generation)
-    /// * `data` - Vector of image frames with delay information
-    ///
-    /// # Returns
-    ///
-    /// Returns `Ok(())` on success or an `Error` on failure.
     pub fn write_webp(cache_dir: &path::Path, url: &str, data: Vec<ImageFrame>) -> Result<()> {
         if data.is_empty() {
             return Err(crate::Error::Generic(
@@ -219,46 +204,28 @@ impl MediaCache {
             create_dir_all(p)?;
         }
 
-        if data.len() == 1 {
-            // Static WebP - use lossless encoding
-            let frame = &data[0];
+        let mut config = webp::WebPConfig::new().or(Err(crate::Error::Generic(
+            "Failed to configure webp encoder".to_owned(),
+        )))?;
+        config.lossless = 1;
+        config.alpha_compression = 0;
+
+        let reference_frame = data.first().expect("Data is not empty");
+        let mut encoder = webp::AnimEncoder::new(
+            reference_frame.image.size[0] as u32,
+            reference_frame.image.size[1] as u32,
+            &config
+        );
+
+        for frame in &data {
             let width = frame.image.size[0] as u32;
             let height = frame.image.size[1] as u32;
-
             let encoder = webp::Encoder::from_rgba(frame.image.as_raw(), width, height);
             let encoded = encoder.encode_lossless();
-
-            std::fs::write(file_path, &*encoded)?;
-        } else {
-            // Animated WebP - encode all frames
-            // Note: The webp crate's animation encoding API is limited
-            // For now, we'll fall back to writing individual frames as a sequence
-            // or use the image crate's approach
-
-            // Create an animated WebP using the webp crate
-            // First, we need to encode each frame
-            let mut encoded_frames = Vec::new();
-
-            for frame_data in &data {
-                let width = frame_data.image.size[0] as u32;
-                let height = frame_data.image.size[1] as u32;
-                let encoder = webp::Encoder::from_rgba(frame_data.image.as_raw(), width, height);
-                let encoded = encoder.encode_lossless();
-                encoded_frames.push((encoded, frame_data.delay));
-            }
-
-            // For animated WebP, we'll use a simple approach:
-            // Write the frames as a GIF-style animation format that browsers support
-            // Unfortunately, the webp crate doesn't have a high-level animated encoder
-            // So we'll fall back to just writing the first frame for now
-            // TODO: Implement proper animated WebP encoding when the crate supports it
-
-            if let Some((first_encoded, _)) = encoded_frames.first() {
-                std::fs::write(file_path, &**first_encoded)?;
-            }
+            encoded_frames.push((encoded, frame.delay));
         }
 
-        Ok(())
+        std::fs::write(file_path, &**first_encoded)?
     }
 
     pub fn key(url: &str) -> String {