From 4c09edb7cfb22a98420f4a11bee6ae5b29525db4 Mon Sep 17 00:00:00 2001
From: Jonas Kalderstam <jonas@cowboyprogrammer.org>
Date: Thu, 6 Jun 2024 22:43:15 +0200
Subject: [PATCH] Fixed too large images on tablets

---
 .../feeder/ui/compose/feedarticle/ReaderView.kt             | 2 +-
 .../feeder/ui/compose/html/LinearArticleContent.kt          | 6 ------
 2 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/app/src/main/java/com/nononsenseapps/feeder/ui/compose/feedarticle/ReaderView.kt b/app/src/main/java/com/nononsenseapps/feeder/ui/compose/feedarticle/ReaderView.kt
index a7964adee..e5c1ec229 100644
--- a/app/src/main/java/com/nononsenseapps/feeder/ui/compose/feedarticle/ReaderView.kt
+++ b/app/src/main/java/com/nononsenseapps/feeder/ui/compose/feedarticle/ReaderView.kt
@@ -313,7 +313,7 @@ fun ReaderView(
                             modifier =
                                 Modifier
                                     .clip(RectangleShape)
-                                    .fillMaxWidth(),
+                                    .width(dimens.maxReaderWidth),
                         ) {
                             WithTooltipIfNotBlank(tooltip = stringResource(id = R.string.article_image)) {
                                 val maxImageWidth by rememberMaxImageWidth()
diff --git a/app/src/main/java/com/nononsenseapps/feeder/ui/compose/html/LinearArticleContent.kt b/app/src/main/java/com/nononsenseapps/feeder/ui/compose/html/LinearArticleContent.kt
index 07514919f..bc057bf9a 100644
--- a/app/src/main/java/com/nononsenseapps/feeder/ui/compose/html/LinearArticleContent.kt
+++ b/app/src/main/java/com/nononsenseapps/feeder/ui/compose/html/LinearArticleContent.kt
@@ -485,12 +485,6 @@ fun LinearImageContent(
                             Modifier
                                 .widthIn(max = maxWidth)
                                 .fillMaxWidth(),
-//                            .run {
-//                                // This looks awful for small images
-//                                dimens.imageAspectRatioInReader?.let { ratio ->
-//                                    aspectRatio(ratio)
-//                                } ?: this
-//                            },
                     )
                 }
             }