Twenty Seventeen: Cannot resize image in the post editor due to a css rule

[mihaivalentin]

WordPress 4.7.2, TwentySeventeen theme, wp-admin, Chrome Windows/Linux latest

• trying to add an image to a post
• try to resize using the resize handles

The image does not resize.

After some investigation, the problem is line 336 of editor-styles.css from the twentyseventeen theme. ​https://github.com/WordPress/WordPress/blob/4.7.2/wp-content/themes/twentyseventeen/assets/css/editor-style.css#L336

The problem is that the comment states that it is for IE8, but it is also used in modern browsers. Removing this property works and makes the image resize possible.

Can you please investigate this?

Thanks!

[RDall]

I reported this on the WordPress Forms and can confirm the bug.

​https://wordpress.org/support/topic/can-not-adjust-image-size-without-caption/

Using WordPress 4.7.4
Twenty Seventeen Version: 1.6
PHP 7.2.7
Tried both Chrome & FireFox, three different installs and two different hosting providers. All resulted in the same issue.

See Screencast:
​https://cloudup.com/ieWsxqRouvq

[audrasjb]

Drop support for IE8 since it's not supported anymore by WordPress Admin and this line is breaking image resizing

[audrasjb]

Removing that line seems to solve the issue

[audrasjb]

Hi, Welcome to WordPress Trac @mihaivalentin :)

Thanks for the issue, it's a good catch!

I tested removing the CSS declaration you quoted, and it seems to work fine.

I'd say we can remove that line since WordPress Admin doesn't support IE8 anymore.

Cheers,

Jb

[swissspidy]

@audrasjb Note that the version field indicates the version where the issue was first reported or introduced.

[audrasjb]

@swissspidy thanks for your feedback and sorry for the mistake.

[laurelfulford]

#45580 was marked as a duplicate.

[laurelfulford]

Thanks for the report, @mihaivalentin, and for the patch, @audrasjb!

This update fixes the issue for me! As mentioned, the TinyMCE editor no longer supports IE8 -- in that browser only the text view is displayed, so this style isn't doing anything.

[laurelfulford]

Adding a note that I made an error in missing that #45580 was a duplicate of this ticket.

@subrataemfluence took the time to create a patch there, and should also get props for the work on this issue (thanks @subrataemfluence!).

[laurelfulford]

In 44205:

Twenty Seventeen: Remove editor style that prevents images from being resized.

Twenty Seventeen’s editor styles contained an Internet Explorer 8 specific fix that stopped images from being scalable in the editor in other browsers. Since the editor no longer supports IE8, this fix can safely be removed.

Props audrasjb, mihaivalentin, subrataemfluence.
Fixes #39738.

[laurelfulford]

Reopening and marking fixed-5.0 so that this can be properly merged into trunk.

[desrosj]

In 44307:

Twenty Seventeen: Bug Fixes.

• Remove unnecessary ol styles from editor. Twenty Seventeen was designed with bolded numbers for its ordered list styles. This was removed from the theme prior to launch, but it was left in the editor styles, so it needed to be removed there, too.
• To improve accessibility, the cookies content checkbox in the comment form needed some visible indication when it’s in focus.
• Twenty Seventeen’s editor styles contained an Internet Explorer 8 specific fix that stopped images from being scalable in the editor in other browsers. Since the editor no longer supports IE8, this fix can safely be removed.
• Fix the font-family used for Simplified Chinese (zh_CN) in the block editor CSS.

Props mmaumio, edpittol, pratikthink, afercia, audrasjb, mihaivalentin, subrataemfluence, vaishalipanchal, ze3kr, laurelfulford.

Merges [44203-44205] and [44235] into trunk.

Fixes #44775, #44699, #39738, #45408.