[UploadImage] Use uploaded.width/height if set

[Piotrek Koszuliński]

Split from #13520. Requires #13518.

The server should be able to return width and height of the image. #13518 will assure that these properties are propagated to the onUploaded callback. In this ticket we need to use these properties (if defined) to set width and height of the resulted image.

[Tomasz Jakut]

Done, using loader.responseData property introduced by #13519. Pushed ​branch:t/13794.

[Tade0]

Added with ​git:59944a6499aa7debf253e5e0edc8361ce95b2ac9.

[Gosh Snobo]

I would propose to reopen this ticket. Currently, onUploaded sets image width & height if non-empty values are returned from server, otherwise it sets $img.naturalWidth & $img.naturalHeight. But I think it should be a valid scenario to NOT set width/height, e.g. to allow for responsive images (those with max-width=100% in the CSS).

In case of traditionally uploaded image, config.image2_prefillDimensions setting serves exactly this purpose. In case of dropped upload, however, this setting has no effect. So imho either the same setting should be also governed by onUploaded, or we need a new setting (though I think it would be more confusing).

[porjo]

Could someone please update the ImageUpload doco here ​http://docs.ckeditor.com/#!/guide/dev_file_upload to mention the ability to set 'width' and 'height' from the backend.