Allow user-specified default image size when inserting image via Press This

[topdownjimmy]

Currently, when adding an image to a post in a Press This window, the image is inserted at full size by default. It would be helpful if the default insertion size could be specified by the user.

This is unrelated to Ticket #30269, as that concerns inserting images in the standard post editor.

[topdownjimmy]

Patch to apply image_default_size setting to Press This insertions

[topdownjimmy]

Patch attached, though there may be a more elegant way of accomplishing this.

[topdownjimmy]

Different method, interferes less with media_sideload_image()

[topdownjimmy]

Fixing is_wp_error check in side_load_images

[topdownjimmy]

Attempting a safer size- img class replacement with this patch

[topdownjimmy]

Curiously, even without my patches, if the user specifies a custom class name in the Press This window, the size-full class name is overwritten (though alignnone remains).

[topdownjimmy]

Better handling of class attribute string

[stephdau]

32137.5.patch should work,but:

• please make sure to build your patches from the repository root: src/...
• if ( $return === 'id' ) and if (get_option('image_default_size') != '' && get_option('image_default_size') != 'full') should use Yoda conditions.
• $src_array[0] should be checked to make sure it has the expected value before using it.
• preg_match( '/(class="[^"]+)"/', should check for both " and ' as attribute delimiters.
• might as well instantiate a variable for get_option('image_default_size')

[topdownjimmy]

Thanks for the feedback @stephdau.

[topdownjimmy]

I think this may be a better approach for validating the output of wp_get_attachment_image_src? I'm not positive.

[topdownjimmy]

FIX: I think this may be a better approach for validating the output of wp_get_attachment_image_src? I'm not positive.

[azaozz]

Couple of notes:

• To do this properly (restrict the inserted image size) we will need to restrict it when first inserted, from JS. Otherwise the user will see the "full" image in Press This, and it will be replaced with the "default" size on saving. That difference can be pretty large, and definitely unexpected.
• The option image_default_size hasn't been used for years. As far as I remember it is a left over from the old media modal, pre 3.5. Perhaps we could use get_user_setting('imgsize') which is the size the user selected when last inserting an image into the editor. However there's no way for the user to change that size in PT.

In that terms the current workflow seems better: insert the full image size (as found on the source site), and replace it with the same size on saving. If the user likes to change it, that can be done from the Edit Post screen in wp-admin, same as the rest of the more advanced settings.

Looking at 32137.7.patch, it seems like a good idea to return the attachment id from media_sideload_image(), then do wp_get_attachment_image() so the image tag has the proper class, etc. Then plugins will be able to use several of the image filters to change size or other attributes for the replacement image tag.

[kraftbj]

With PT being moved into a plugin, moving the conversation to ​https://github.com/WordPress/press-this/issues/5