[Feature]: Adjustable image sizes

[dexdeadly]

Short Feature Title

Image size adjustments

What area does this affect?

QuickBar Overlay

The Problem

I previously had a notification with a gif that would show at the bottom of the TV along with the text from a different notification setup. However I noticed no matter what size image I use when the notification shows up its a good 30% of the screen I'd say. Would be nice if i could choose how big the image is in the notification so that its subtle.

The Proposed Solution

Allow the ability to change the size of image when notification displays on screen.

Screenshots or Video

No response

Visuals / Mockups

No response

Additional Information

N/A

Home Assistant Context (Optional)

N/A

Readiness Check

• I have confirmed this isn't already a feature/setting in the app by also searching the guide.
• I understand that this is an open-source project managed alongside a full-time job and B.Sc. studies by Omri, and PRs and Issues may take some time.
• I am willing to help provide more information upon request, and test beta builds of this feature.

[Trooped]

Hey @dexdeadly,
it's definitely not adjustable right now - it works like this:
Image height maximum value = 200dp
Image width minimum value = 200dp
image width maximum value = 600dp

There's an automatic calculation that happens based on the image provided that 'decides' the actual image size, and it's done logically, with the constraint that the entire image must be shown. This is the code in this file:

var actualAspect by remember(imageUrl) { mutableStateOf<Float?>(null) }
    val initialWidthGuess = (imageMaxHeight * EXPECTED_ASPECT_GUESS).coerceIn(minCardWidth, maxCardWidth)
    val targetWidth = remember(actualAspect, maxCardWidth, imageMaxHeight) {
        val a = (actualAspect ?: EXPECTED_ASPECT_GUESS).coerceAtLeast(0.25f) // guard against 0 or nonsense
        (imageMaxHeight * a).coerceIn(minCardWidth, maxCardWidth)
    }
    val animatedCardWidth by animateDpAsState(
        targetValue = if (hasImage) targetWidth else minCardWidth,
        animationSpec = tween(220),
        label = "cardWidthAnimV2"
    )

I'm not sure it's a very wanted feature, so I'll see what others say, as well as suggestions some people may have on how it should be handled.