Twenty Seventeen: images in the Classic Block cannot be resized

[azaozz]

To reproduce:

1. Start a new post.
2. Insert a Classic Block.
3. Insert an image.
4. Attempt to resize it by dragging the corners.

The image "snaps back" to 100% width.

Seems that has been the case for a while (in the Classic Editor). Caused by width: inherit; from here:

.editor-styles-wrapper img {
    height: auto; /* Make sure images are scaled correctly. */
    width: inherit; /* Make images fill their parent's space. Solves IE8. */
    max-width: 100%; /* Adhere to container width. */
}

(which is imported from editor-style.css).

[subrataemfluence]

Proposed patch

[subrataemfluence]

Please ignore the previous patch

[subrataemfluence]

I have created a proposed patch using your suggestion.
With width: inherit; not included, I am able to resize the image.

Please see the screencast ​https://youtu.be/EB9yt_Ew-d8

[laurelfulford]

Thanks for spotting this, @azaozz, and for the fix, @subrataemfluence!

I've confirmed that this update does take care of the issue in the Classic Block and the TinyMCE editor.

The width: inherit; was originally a fix for an IE8 issue; from what I can see, TinyMCE stopped supporting IE8, and I can only see the text tab with no option to switch to visual when I test that browser in Browserstack. So these styles should no longer be needed at all in the editor, either. But just in case -- @azaozz, could you confirm that what I'm seeing for IE8 sounds right for the current support, in case I'm missing something? Thanks!

[azaozz]

fix for an IE8 issue...

Ah, both core and TinyMCE don't support IE < 11. Think we can just ignore that fix for inside the editor :)

[azaozz]

can only see the text tab with no option to switch to visual

Me too, emulating in IE11 (which as far as I remember is quite accurate, almost "native").

[laurelfulford]

Thank you for double-checking that, @azaozz! I thought you might just know off the top of your head, so I really appreciate the testing! :)

Based on that, I'd say this fix is ready to roll!

[laurelfulford]

I completely missed that this is a duplicate of an existing ticket, #39738. :facepalm:

Apologies! Closing this one in favour of that one.