Expose `RelativeContentLocation` as `ZoomableContentLocation.relative` about telephoto HOT 8 CLOSED

… gave a weirdly scaled image in case the image is smaller than screen resolution.

Interesting. ZoomableImage uses scaledInsideAndCenterAligned to support this same use case -- images that are smaller than layout bounds. Can you share some code, the images you're using and your device resolution?

from telephoto.

In case of ZoomableImage the underline Image's content scale is ContentScale.Inside using ContentScale.Inside with scaledInsideAndCenterAligned doesn't produce any issue but in my case i want to use ContentScale.Fit which with scaledInsideAndCenterAligned and an image smaller than screen resolution makes the displayed image weirdly zoomed in. In my case I tested with the small image below on a 720x1600 resolution device

from telephoto.

@saket any update?

from telephoto.

from telephoto.

I tried out the above image on a 720x1600 display but I'm unable to reproduce the problem. @AntsyLich please share a screenshot and some code? A reproducer project would be the best if you've got time.

ZoomableAsyncImage(
  model = R.drawable.check_pattern,
  contentDescription = …,
  contentScale = ContentScale.Fit,
)

from telephoto.

I also used.

var canvasSize by remember { mutableStateOf(Size.Zero) }
LaunchedEffect(canvasSize) {
    if (canvasSize == Size.Zero) return@LaunchedEffect
    zoomState.setContentLocation(
        ZoomableContentLocation.scaledInsideAndCenterAligned(),
    )
}

If you still can't reproduce I'll get a reproducible code up

from telephoto.

If you still can't reproduce I'll get a reproducible code up

Yes please! The code snippet you shared is incomplete. It doesn't show how canvasSize is populated.

I'm also curious to know why you aren't using ZoomableImage / ZoomableAsyncImage directly? Is that because they are incorrectly scaling your image?

from telephoto.

FWIW I've added ZoomableContentLocation.scaledToFitAndCenterAligned(), but you shouldn't need to use it if you're using ZoomableAsyncImage() / ZoomableGlideImage().

It'll be available in the next release.

from telephoto.