1465238 - Srcset is ignored in summary view of RSS feed message

Status: UNCONFIRMED

(Core :: Layout: Images, Video, and HTML Frames, defect)

Description

[agentmc]

Attachment: Capture3.png

User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.183 Safari/537.36 Vivaldi/1.97.1183.3

Steps to reproduce:

1. Subscribe to Colossal feed: http://www.thisiscolossal.com/feed/
2. Make sure settings for the message view are: Summary (because you don't want to load entire Colossal which makes little sense to do anyway) and Original Html (because you want images + description - whole site is about it)
2. Select message in messages list


Actual results:

The images in Message view are following the Width and Height attributes in the inline HTML effectively ignoring Srcset attribute. As the result, you have 2 options: zoom to 100% to read text and zoom to ~25% to see the image (I have 15" 1366x768 display).


Expected results:

There are couple of options:
1) follow the standard (as far as I see it): Srcset is respected; the image width is taken based on viewbox width (closest available). This approach would probably result in the image being still wider than viewbox but at least bearably wider.
2) follow the logic used in M2 Opera's viewer: take the most appropriate img from srcset and scale it to 100% width. That would be the approach I 'd like to have implemented even though it doesn't follow the standard.
3) I'm not an expert in RSS, maybe there is somewhere CSS being linked? In that case just follow the definition of 'full-width' class defined in html being width:100%.

Comment 1

[Jorg K (CEST = GMT+2)]

What happens when you save the message as HTML and view the HTML file in Firefox?

Comment 2

[agentmc]

The page is rendered the same way wrongly.
Chrome renders it better (smaller image) but also does not scale to 100% screen.
It seems at the moment the only thing capable of rendering the message properly is the old Opera 12's reader.

Comment 3

[agentmc]

Attachment: A New Three-Dimensional Installation by Chris Engman Invites the Viewer to Step Inside a Photograph.html

Attaching the saved page

Comment 4

[Jorg K (CEST = GMT+2)]

Thanks! Off it goes to Firefox since TB is using the Mozilla rendering engine. Guessing that Core::Layout might be the right product.

Comment 5

[Emilio Cobos Álvarez (:emilio)]

I don't think this is valid as a Layout bug per se. width="" and height="" attributes override the size of the image, and this happens in other browsers as well.

Can you point out the bit of spec that you thing changes what the expected rendering should be?

Probably the message reader could set max-width or something of the sort to prevent images from overflowing.

Comment 6

[agentmc]

If the question is to me @Emilio then I can't answer because I'm not HTML5 spec expert. But here they say that width will not override the srcset, but rather work as a temporary solution until browser chooses the right value from srset: https://html.com/attributes/img-srcset/#Speed_Up_Rendering_with_sizes : "You can let the browser wait to figure this out (once it has parsed all the CSS), or you can give it enough information ahead of time so that it can start loading the correct image right away. This information can be provided with the width and height attributes."

Comment 7

[Emilio Cobos Álvarez (:emilio)]

Well, that's just wrong. width="" and height="" doesn't change the image that gets loaded in any way.