Coil ImageView doesn't fit exactly

Question

According to Coils docs, I don't have to make any configuration for my image to fit(). The problem is, that the ImageView is not loading correctly:

This is my configuration for the ImageView with Picasso:

picasso.load(unsplashResult?.photoUrls?.thumbnailUrl)
                        .fit()
                        .error(R.drawable.img_placeholder)
                        .placeholder(R.drawable.img_placeholder)
                        .into(currentImageView)

And this is the code with Coil:

currentImageView.load(unsplashResult?.photoUrls?.thumbnailUrl) {
                        placeholder(R.drawable.img_placeholder)
                        error(R.drawable.img_placeholder)
                    }

The result on my ImageView is not the same though.

This is how Coil loads them:

And this is how Picasso loads them:

Question is, how can I achieve the same result with Coil?

EDIT

This is my ImageView:

<ImageView
            android:adjustViewBounds="true"
            android:id="@+id/ivUnsplashImage"
            android:src="@drawable/some_unsplash_img"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"/>

hmmm. actually it says that there is no need to make any fit configurations. https://coil-kt.github.io/coil/migrating/ — coroutineDispatcher, Aug 19 '19 at 09:46

score 8 · Answer 1 · answered Sep 05 '19 at 07:03

8

Based on this document

You can work around this by setting the scaleType to centerCrop or scale(Scale.FILL) on your request.

https://coil-kt.github.io/coil/api/coil-base/coil.request/-request-builder/scale/

https://coil-kt.github.io/coil/api/coil-base/coil.size/-scale/

you are set Scale type this way

view.load(chatMessageBean.thumb) {
       crossfade(750)
       placeholder(errorPlaceHolder)
       transformations(CircleCropTransformation())
       error(errorPlaceHolder)
       scale(Scale.FILL)
}

answered Sep 05 '19 at 07:03

Mayur Patel

2,300
11
30

just one thing, this behavior does resolve, but on second load it still persists – coroutineDispatcher Sep 06 '19 at 13:01
@coroutineDispatcher Can you elaborate more in details? – Mayur Patel Sep 09 '19 at 05:11
well this piece of code does work, but if I click the same fragment again, the photos will get back to the previous state (the not correct one, picture 1 in the question) – coroutineDispatcher Sep 09 '19 at 07:07

score 2 · Answer 2 · answered Aug 19 '19 at 09:54

2

So, according to migration guide answer should be like;

imageView.scaleType = ImageView.ScaleType.FIT_CENTER

or it should be defined in XML

android:scaleType="fitCenter"

answered Aug 19 '19 at 09:54

Ferhat Ergün

135
1
10

1

Nope, same result – coroutineDispatcher Aug 19 '19 at 09:59
did you check this? https://github.com/coil-kt/coil/issues/20 i think there is an issue with scaling – Ferhat Ergün Aug 19 '19 at 10:14
@coroutineDispatcher Looks like this is related to this issue (https://github.com/coil-kt/coil/issues/20). I'm working on fixing it for the 0.7.0 release in a couple days. – Colin White Aug 25 '19 at 05:49
yes, but in my case it still doesn't give the expected result, even after setting the `scaletype` to `centercrop` – coroutineDispatcher Aug 26 '19 at 07:21

RatneZ · Answer 3 · 2020-11-02T08:48:51.517

1

As much I understood from the documentation you should use ImageView.ScaleType

imageView.scaleType = ImageView.ScaleType.FIT_XY

or you can use it into your xml tag of your ImageView

android:scaleType="fitXY"

Apart from FIT_XY you can use ImageView.ScaleType.CENTER_CROP too according to your needs.

edited Nov 02 '20 at 08:48

answered Nov 02 '20 at 08:15

RatneZ

1,078
9
9

score 0 · Answer 4 · edited Mar 27 '20 at 02:56

0

Just remove the line android:adjustViewBounds="true" in your layout XML file.

edited Mar 27 '20 at 02:56

Robert Columbia

6,313
15
32
40

answered Mar 27 '20 at 02:44

Quý Vương

1

Coil ImageView doesn't fit exactly

4 Answers4