Scaling prompt below uploader

[jane]

In the uploader window, it says:
"Scale images to match the large size selected in image options (1024 × 1024)."

While many people upload giant iPhone or camera images that could benefit from being scaled down automatically, even more upload images that are nowhere near that big. This line suggests smaller images will be scaled up in that case, which is not good.

1. Is it going to scale things up as well?

2. Is it scaling down and only uploading the scaled down version, or is it still uploading the full size? If someone chooses Link to: Original file, would they link to the 'large size' or the original size?

Am concerned about this feature being confusing.

[johnbillion]

+1. It's not at all clear what this option does.

[DrewAPicture]

What about something like this:

[] Scale down images with dimensions larger than 1024 x 1024 (the large size selected in image options).

Also, I remember there being brief discussion about a shorter label followed by more of a description. An example:

[] Scale down large images

Images with dimensions higher than the large size selected in image options (1024 x 1024) will be proportionally resized to fit.

Or something.

[azaozz]

Replying to jane:

1. Is it going to scale things up as well?

No.

2. Is it scaling down and only uploading the scaled down version, or is it still uploading the full size? If someone chooses Link to: Original file, would they link to the 'large size' or the original size?

This depends on the uploader runtime and browser used. HTML5 (the default runtime) would scale the images before uploading in FF and Chrome but not in Opera or Safari. Silverlight or Flash (used in IE) will scale before uploading: ​http://www.plupload.com/

Perhaps something like the second option suggested by @DrewAPicture would be better:

[] Scale down large images

Images will be proportionally resized to the large size selected in <a>image options</a>
(1024 x 1024) if they are larger. 

[DrewAPicture]

Here's a patch based on azaozz's suggested wording

[jane]

If the action is not consistent between browsers, I'm uncomfortable with making any statement about what's happening, unless we're sniffing and showing specific message (which seems excessive for something people may or may not even want to use. Wondering if we should pull the auto-scale out altogether and make it a plugin.

[SergeyBiryukov]

FWIW, there was a relevant ​IRC discussion in October.

[DrewAPicture]

Replying to jane:

If the action is not consistent between browsers, I'm uncomfortable with making any statement about what's happening, unless we're sniffing and showing specific message (which seems excessive for something people may or may not even want to use. Wondering if we should pull the auto-scale out altogether and make it a plugin.

On the one hand, I see this as very valid functionality. I manually re-scaled images on a newspaper website the other week for 2 years worth of uploads and dropped the filesizes by 75 percent. So having this at the outset would provide a certain performance upgrade.

On the other hand, since it's not consistent cross-browser, I do see the value in browser sniffing to display or not display, but I agree that it seems a bit "excessive" for something to include in core. Maybe pushing out to a separate plugin would be a good idea.

[ryan]

Per bug scrub, we'll remove this from 3.3, make it a plugin, and look at it again in 3.4.

[azaozz]

In [19223]:

Remove the Resize files checkbox for now (revisit in 3.4), fixes #19174

[nacin]

What if we left all of [19223] but just removed the hook? Makes it a heck of a lot easier for a plugin to implement it in all it's half-baked glory.

[azaozz]

Replying to nacin:

Makes sense, will revert in a min.

[DrewAPicture]

Replying to nacin:

What if we left all of [19223] but just removed the hook? Makes it a heck of a lot easier for a plugin to implement it in all it's half-baked glory.

+1

[azaozz]

In [19225]:

Revert [19223] and only stop showing the checkbox, props nacin, see #19174

[azaozz]

In [19226]:

Revert media.php too, see #19174

[uggbootsspace]

Nothing.

[jayarjo]

I can help to overcome inconsistencies here, if any. But I'm not sure what the requirements for client-side image resizing should be. Is it meant to upsize too?

Having certain combination of options and narrowing runtime choice to HTML5, Flash and SilverLight only (other runtimes cannot scale images and/or retain EXIF information #14459), it can be made to operate consistently across vast majority of available platforms (if you could name problematic ones I could take care of them too).

[DrewAPicture]

Replying to jayarjo:

Is it meant to upsize too?

I hope not, scaling images up generally doesn't produce the best results just as attempting to make standard def video magically into high def video doesn't really work out.

it can be made to operate consistently across vast majority of available platforms

I think we'll just have to test to see which browsers/platform work and which don't then allow/disallow based on that. But as Jane said before, the labeling that goes with this would need to change depending on the expected behavior of those browsers/platforms.