Small fixes for video attachments

commetchat · Jul 3, 2024 · 01fc3fc · 01fc3fc
1 parent d89fea2
commit 01fc3fc
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 2 deletions.
diff --git a/commet/lib/ui/molecules/video_player/video_player_implementation.dart b/commet/lib/ui/molecules/video_player/video_player_implementation.dart
@@ -111,6 +111,6 @@ class _VideoPlayerImplementationState extends State<VideoPlayerImplementation> {
     }
 
     return Size(
-        player.state.height!.toDouble(), player.state.width!.toDouble());
+        player.state.width!.toDouble(), player.state.height!.toDouble());
   }
 }
diff --git a/commet/lib/ui/organisms/attachment_processor/attachment_processor.dart b/commet/lib/ui/organisms/attachment_processor/attachment_processor.dart
@@ -61,6 +61,12 @@ class _AttachmentProcessorState extends State<AttachmentProcessor> {
     super.initState();
   }
 
+  @override
+  void dispose() {
+    videoController?.pause();
+    super.dispose();
+  }
+
   void loadExif() async {
     late Map<String, IfdTag> data;
     if (widget.attachment.data != null) {

diff --git a/tiamat/lib/atoms/panel.dart b/tiamat/lib/atoms/panel.dart
@@ -83,7 +83,11 @@ class Panel extends StatelessWidget {
               padding: const EdgeInsets.fromLTRB(8, 8, 0, 0),
               child: Align(
                   alignment: Alignment.centerLeft,
-                  child: tiamat.Text.labelLow(header!)),
+                  child: tiamat.Text.labelLow(
+                    header!,
+                    maxLines: 1,
+                    overflow: TextOverflow.ellipsis,
+                  )),
             ),
           if (header != null)
             Padding(