Display compression type for images and video

src/audio.rs

@@ -198,10 +198,10 @@ impl Content for Audio {
       )
       .value("channels", self.channels)
       .value(
-        "compression mode",
+        "compression",
         match self.ty {
-          AudioType::Flac => "lossless",
-          AudioType::Mp3 => "lossy",
+          AudioType::Flac => Compression::Lossless,
+          AudioType::Mp3 => Compression::Lossy,
         },
       )
       .value("samples", self.samples)
@@ -464,7 +464,7 @@ mod tests {
         .value("sample rate", "44.1 kHz")
         .value("bit rate", "4 kbit/s")
         .value("channels", "2")
-        .value("compression mode", "lossless")
+        .value("compression", "lossless")
         .value("samples", "66150")
         .build(),
     );
@@ -492,7 +492,7 @@ mod tests {
         .value("sample rate", "44.1 kHz")
         .value("bit rate", "4 kbit/s")
         .value("channels", "2")
-        .value("compression mode", "lossy")
+        .value("compression", "lossy")
         .value("samples", "66150")
         .build(),
     );
@@ -514,7 +514,7 @@ mod tests {
         .value("type", "FLAC")
         .value("sample rate", "0 kHz")
         .value("channels", "2")
-        .value("compression mode", "lossless")
+        .value("compression", "lossless")
         .value("samples", "0")
         .build(),
     );

src/compression.rs

@@ -0,0 +1,8 @@
+use super::*;
+
+#[derive(Display)]
+#[strum(serialize_all = "kebab-case")]
+pub(crate) enum Compression {
+  Lossless,
+  Lossy,
+}

src/image.rs

@@ -278,6 +278,13 @@ impl Content for Image {
       .value("bit depth", format!("{}-bit", self.bit_depth))
       .optional("chroma subsampling", self.chroma_subsampling)
       .value("alpha", self.alpha)
+      .value(
+        "compression",
+        match self.ty {
+          ImageType::Jpeg => Compression::Lossy,
+          ImageType::Png => Compression::Lossless,
+        },
+      )
   }
 
   fn load(root: &Utf8Path, path: RelativePath) -> Result<Item<Self>> {
@@ -445,6 +452,22 @@ mod tests {
     assert!(!root.join("thumbnails/foo.jpg").exists());
   }
 
+  #[test]
+  fn info() {
+    assert_eq!(
+      Content::info(&Image::test("foo.jpg"), InfoBuilder::new()).build(),
+      InfoBuilder::new()
+        .value("type", "JPEG")
+        .value("dimensions", "1×1")
+        .value("orientation", "0°")
+        .value("color type", "RGB")
+        .value("bit depth", "8-bit")
+        .value("alpha", "false")
+        .value("compression", "lossy")
+        .build(),
+    );
+  }
+
   #[test]
   fn load() {
     #[track_caller]

src/item.rs

@@ -108,6 +108,7 @@ mod tests {
         .value("color type", "RGB")
         .value("bit depth", "8-bit")
         .value("alpha", "false")
+        .value("compression", "lossless")
         .build(),
     );
 
@@ -124,6 +125,7 @@ mod tests {
         .value("color type", "RGB")
         .value("bit depth", "8-bit")
         .value("alpha", "false")
+        .value("compression", "lossless")
         .build(),
     );
   }

src/lib.rs

@@ -47,6 +47,7 @@ use {
     color_type::ColorType,
     component::Component,
     component_error::ComponentError,
+    compression::Compression,
     content::Content,
     content_type::ContentType,
     context::Context,
@@ -312,6 +313,7 @@ mod color_type;
 mod component;
 mod component_buf;
 mod component_error;
+mod compression;
 mod content;
 mod content_type;
 mod context;

src/templates/media.rs

@@ -117,6 +117,12 @@ mod tests {
                           false
                         </dd>
                       </div>
+                      <div>
+                        <dt>compression</dt>
+                        <dd>
+                          lossless
+                        </dd>
+                      </div>
                     </dl>
                   </li>
                 </ol>

src/video.rs

@@ -47,6 +47,7 @@ impl Content for Video {
         "duration",
         DisplayDuration(Duration::from_millis(self.duration)),
       )
+      .value("compression", Compression::Lossy)
       .list(
         "tracks",
         self
@@ -119,6 +120,19 @@ mod tests {
     });
   }
 
+  #[test]
+  fn info() {
+    assert_eq!(
+      Content::info(&Video::test("foo.mp4"), InfoBuilder::new()).build(),
+      InfoBuilder::new()
+        .value("type", "MP4")
+        .value("duration", "0:01")
+        .value("compression", "lossy")
+        .list("tracks", Vec::new())
+        .build(),
+    );
+  }
+
   #[test]
   fn load() {
     #[track_caller]