volto
volto copied to clipboard
plone
Image block large option should point to large scale
Is your feature request related to a problem? Please describe. From this code of the Image block: https://github.com/plone/volto/blob/master/src/components/manage/Blocks/Image/Edit.jsx#L280
We can see the following: S option gives us mini scale M option gives us preview scale L option gives us the original image
Describe the solution you'd like We should modify the large option to use the large scale. I would propose to have an extra "O" option for original image size which takes over the behavior of the "L" option that currently references the original image
I think it's conflating two issues and the problem is complex.
"Size" in the image block is relevant to how the image is rendered in the page, not to the scale that's grabbed from the server. In this case, the Original and Large would have no apparent "different" visual effect for the user.
I think the UI should be like this: options S/M/L which would all use the scales. When L is activated, show an extra checkbox option, "Use best quality (slower speed)", which would bypass the URL scale.
I think it's conflating two issues and the problem is complex.
"Size" in the image block is relevant to how the image is rendered in the page, not to the scale that's grabbed from the server. In this case, the Original and Large would have no apparent visual effect for the user.
I think the UI should be like this: options S/M/L which would all use the scales. When L is activated, show an extra checkbox option, "Render image in maximum quality (slower download)", which would bypass the URL scale. I wonder if people will notice that extra checkbox or think of enabling it.
I am still in favor of introducing an extra option so that we fix the inconsistency of having a large size not reference an actual scale size and thus introducing potential performance issues without even realizing it.
@sneridagh what is your opinion on this issue? I could tackle this issue myself if there is a consent on what we would like to have.
@plone/volto-team We need a fix for this, it looks like /@@images/image no longer works with Plone 6.
See https://github.com/plone/volto/blob/4a760bde9bc2ea8050d7a16f2b518e244d415536/src/components/manage/Blocks/Image/View.jsx#L45