[RSS Reader] Image size in article view

[Lisette Slegers]

Images in the article view are not full width.

---
Desired solution

Can we scale up the image so that they are the full width of the screen, as per page 6 of the visual spec https://docs.google.com/a/canonical.com/file/d/0BzWzYe656QsATEoxSjQ4WUVDRTg/edit

Scaling up images will result in lower quality, but it seems to be fine in Feedly (see attachment).

[Roman Shchekin (mrqtros)]

Let me explain my opinion.

We have two kind of bugs (related to images) in our atricle view: some images are too small, some too big.
Let's speak about small first. Content of RSS article is a pure HTML, where feed's owner can display anything what he wants to user. And he must have ability to display tiny smile or twitter icon. If we stretch the images to the full width, the application will become unusable. So I propose don't touch little images.

But sometimes article contain <img> tag with image, which width is bigger than view width. Unfortunately, we can't control width of such images (limit their width) since they are inside Label component. I see only one solution - use online service to resize image. Their usage always similar to
service.url.com/<params>/<original_url>
It means that all images will go through proxy server, which will transform them as needed.

What do you think about that solution?
Screenshots of usage in Shorts can be found here: http://ubuntuone.com/25gpFDvDQSfhmul6twtfZU

[Lisette Slegers (lisette-slegers)]

What is the status of this bug? Can we successfully guess which image is an article image (instead of a twitter icon)? Can we stretch the image over the full width of the page?

[David Planella (dpm)]

- Small images: I agree with Roman, let's not modify small images. In fact, IIRC we already ignore them if they are smaller than a given size?

- Bigger images: as discussed in another thread, I think we should not make Shorts dependent of a 3rd party service for resizing, especially as that would also add a delay to displaying single articles. It seems that the Label component is constraining us a bit here, could we not use another component or a workaround to resize images to fit the width of the screen? I know you're all very clever guys, does anyone have any ideas how we could achieve that?

[Roman Shchekin (mrqtros)]

Without downloading image to file we can only use heuristics to determine
must we stretch image to fullscreen or no.
I can only propose us use my "show media" implementation, which uses images
from special data source in article. And this source often contains size of
image (whoohoo), which we only need, BUT - articles (all I've seen before)
 contain small variant of attached image in their body too, and seems there
are no easy way to determine that they are identical, but different size
(to replace little one with big one).
In my opinion RSS Reader should display article exactly as it was formed.
It's like tiny browser. What if browser will change your html page? :)

But I'll try something anyway :)

2013/10/14 David Planella <email address hidden>

> - Small images: I agree with Roman, let's not modify small images. In
> fact, IIRC we already ignore them if they are smaller than a given size?
>
> - Bigger images: as discussed in another thread, I think we should not
> make Shorts dependent of a 3rd party service for resizing, especially as
> that would also add a delay to displaying single articles. It seems that
> the Label component is constraining us a bit here, could we not use
> another component or a workaround to resize images to fit the width of
> the screen? I know you're all very clever guys, does anyone have any
> ideas how we could achieve that?
>
> --
> You received this bug notification because you are a member of Ubuntu
> RSS Feed Reader Developers, which is subscribed to Ubuntu RSS Feed
> Reader App.
> Matching subscriptions: Ubuntu RSS Reader Bugmail
> https://bugs.launchpad.net/bugs/1215357
>
> Title:
> [RSS Reader] Image size in article view
>
> Status in RSS Feed Reader application for Ubuntu devices:
> Triaged
> Status in Ubuntu UX bugs:
> Fix Committed
>
> Bug description:
> Images in the article view are not full width.
>
> ---
> Desired solution
>
> Can we scale up the image so that they are the full width of the
> screen, as per page 6 of the visual spec
>
> https://docs.google.com/a/canonical.com/file/d/0BzWzYe656QsATEoxSjQ4WUVDRTg/edit
>
> Scaling up images will result in lower quality, but it seems to be
> fine in Feedly (see attachment).
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/ubuntu-rssreader-app/+bug/1215357/+subscriptions
>