Automattic / stories-android

Loop concept app - WP Stories library
GNU General Public License v2.0
20 stars 6 forks source link

Improve image quality #566

Open megsfulton opened 4 years ago

megsfulton commented 4 years ago

Is there anything we can do to improve the quality of the image exports?

I didn't expect the text to look fuzzy. These stories were created on a pixel with image compression turned off.

https://wpstories.wordpress.com/2020/10/12/hiking-in-the-southwest/

https://wpstories.wordpress.com/2020/10/12/patagonia-2/

aforcier commented 4 years ago

Confirmed this, adding really big text with background color helps make the issue obvious. It's present in the locally saved images as well. cc @mzorz

megsfulton commented 4 years ago

Another piece of information for looking into this - the image quality for the export to a self-hosted JP site appears to be much higher quality.

https://wpstoriesdemo.mystagingwebsite.com/2020/10/13/image-opt-off/

That story was created on the same build and same device as the two wpstories.wordpress.com stories.

megsfulton commented 3 years ago

Another nugget of info that might mean this is a combo issue with the web viewer as well?

In this example the story output appears fuzzy/pixelated but the source image is not.

Story: https://wpstories.wordpress.com/2020/12/02/story-demo-01/

Image source in the media lib: https://wpstories.files.wordpress.com/2020/12/wp_story1606945178509_0.jpg

aforcier commented 3 years ago

I believe the main issue here (especially where the quality was higher for a Jetpack site vs a WordPress.com site) was resolved with https://github.com/Automattic/jetpack/pull/18609.

However it seems like there may have also been a less pronounced issue with the media being exported from Android from the comments above. I'll leave this open for now and see if we notice any more degraded quality issues.