From 53c73c2195f1eae7c4702956505fcbf198c1587a Mon Sep 17 00:00:00 2001
From: Joseph Donofry <rubberduckie3554@gmail.com>
Date: Wed, 5 Jan 2022 23:33:08 -0500
Subject: [PATCH] Fix some minor UI issues with ImageOverlay and
 RawMessageDialog

---
 resources/qml/dialogs/ImageOverlay.qml     | 49 +++++++++++-----------
 resources/qml/dialogs/RawMessageDialog.qml |  2 +
 2 files changed, 27 insertions(+), 24 deletions(-)

diff --git a/resources/qml/dialogs/ImageOverlay.qml b/resources/qml/dialogs/ImageOverlay.qml
index d7ea7b0e3..2e1261d3e 100644
--- a/resources/qml/dialogs/ImageOverlay.qml
+++ b/resources/qml/dialogs/ImageOverlay.qml
@@ -44,6 +44,7 @@ Window {
             fillMode: Image.PreserveAspectFit
             smooth: true
             mipmap: true
+            property bool loaded: status == Image.Ready
         }
 
         MxcAnimatedImage {
@@ -57,30 +58,30 @@ Window {
         }
 
         BoundaryRule on scale {
-            enabled: img.loaded
+            enabled: img.loaded || mxcimage.loaded
             id: sbr
             minimum: 0.1
-            maximum: 100
-            minimumOvershoot: 0.02; maximumOvershoot: 100
+            maximum: 10
+            minimumOvershoot: 0.02; maximumOvershoot: 10.02
         }
 
-        //BoundaryRule on x {
-        //    enabled: img.loaded
-        //    id: xbr
-        //    minimum: -100
-        //    maximum: imageOverlay.width - img.width + 100
-        //    minimumOvershoot: 100; maximumOvershoot: 100
-        //    overshootFilter: BoundaryRule.Peak
-        //}
-
-        //BoundaryRule on y {
-        //    enabled: img.loaded
-        //    id: ybr
-        //    minimum: -100
-        //    maximum: imageOverlay.height - img.height + 100
-        //    minimumOvershoot: 100; maximumOvershoot: 100
-        //    overshootFilter: BoundaryRule.Peak
-        //}
+        BoundaryRule on x {
+           enabled: img.loaded || mxcimage.loaded
+           id: xbr
+           minimum: -100
+           maximum: imageOverlay.width - img.width + 100
+           minimumOvershoot: 100; maximumOvershoot: 100
+           overshootFilter: BoundaryRule.Peak
+        }
+
+        BoundaryRule on y {
+           enabled: img.loaded || mxcimage.loaded
+           id: ybr
+           minimum: -100
+           maximum: imageOverlay.height - img.height + 100
+           minimumOvershoot: 100; maximumOvershoot: 100
+           overshootFilter: BoundaryRule.Peak
+        }
 
         PinchHandler {
             onActiveChanged: if (!active) sbr.returnToBounds();
@@ -92,10 +93,10 @@ Window {
         }
 
         DragHandler {
-            //onActiveChanged: if (!active) {
-            //    xbr.returnToBounds();
-            //    ybr.returnToBounds();
-            //}
+            onActiveChanged: if (!active) {
+               xbr.returnToBounds();
+               ybr.returnToBounds();
+            }
         }
 
         HoverHandler {
diff --git a/resources/qml/dialogs/RawMessageDialog.qml b/resources/qml/dialogs/RawMessageDialog.qml
index 5da550a69..34104394e 100644
--- a/resources/qml/dialogs/RawMessageDialog.qml
+++ b/resources/qml/dialogs/RawMessageDialog.qml
@@ -38,6 +38,8 @@ ApplicationWindow {
             readOnly: true
             textFormat: Text.PlainText
 
+            anchors.fill: parent
+
             background: Rectangle {
                 color: Nheko.colors.base
             }
-- 
GitLab