background image is stretched, not on iOS

[j2l]

Comparing portrait/landscape body background url behavior on Android and iOS, we can see Android stretches the image, while iOS respects aspect ratio (crop it).
Not sure it needs a sie or distribution parameter.

[realitix]

Hello,
I saw that on Android, background is streched and that looks very ugly. To resolve it, you must provide all aspect ratio and detect screen size. It's better to crop the image.

@gliechtenstein I think it's near this place: https://github.com/Jasonette/JASONETTE-Android/blob/develop/app/src/main/java/com/jasonette/seed/Core/JasonViewActivity.java#L1445

On Glide documentation, they say to use transform : http://bumptech.github.io/glide/doc/transformations.html.

I think it should be something like that:

RequestOptions options = new RequestOptions();
options.centerCrop();

with(JasonViewActivity.this)
   .load(JasonImageComponent.resolve_url(c, JasonViewActivity.this))
   .apply(options)
   .into(new SimpleTarget<GlideDrawable>() {
                                           @Override
                                           public void onResourceReady(GlideDrawable resource, GlideAnimation<? super GlideDrawable> glideAnimation) {
                                               sectionLayout.setBackground(resource);
                                           }    
                                       });  

I can't test on Iphone but behavior must be the same between both.
I will test it when my dev environment will be set up. Currently, I'm testing with the Jason APP, so smooth!

[j2l]

@realitix Merci !
je le ferme.

[realitix]

Why @j2l ? It's not fixed. Can you reopen it ?

[j2l]

Si tu veux JS. On verra.

[realitix]

Merci @j2l, comment sais-tu que je parle Français ?

@gliechtenstein My example can't work because it's with Glide 4 and currently, the project uses Glide3.
I let you know when it's done. My dev-env is ready.

[j2l]

Because your first name is Jean-Sébastien and you live in Paris :)
It was just a guess.

[realitix]

Pas bête !

[gliechtenstein]

Thanks to @realitix this has been resolved!! #119 (comment)