4.9.1 Bug Found in TinyMCE Visual/Text Tab

[othellobloke]

I raised my question here: ​https://wordpress.org/support/topic/switching-between-visual-text-tab-inserts-invisible-span/

When we type something in the visual editor and then click on the Text tab, it inserts the following:

<span style="display: inline-block; width: 0px; overflow: hidden; line-height: 0;" data-mce-type="bookmark" class="mce_SELRES_start"></span>

and then when switching back to Visual it erases what we wrote. But then when we switch back to the Text tab, the span I pasted above keeps getting appended to the one already there.

[azaozz]

Hi @othellobloke, from the support forum thread seems this was an issue with a specific theme that has been resolved. Closing as invalid, feel free to reopen with more info and steps to reproduce on a clean install.

[lizkarkoski]

This error also occurs in WordPress.com. I was able to reproduce on a test site using Twenty Seventeen following up on user reports that came to me in the forums: ​http://en.forums.wordpress.com/topic/change-speed-of-slide-show-in-dara-theme

WordPress.com themes affected: Dara, Twenty Seventeen (I did not test additional WP.com themes)

To reproduce

• add the shortcode for the image slideshow in to a new post. It doesn't matter if you use Calypso or WP-Admin, as both are affected.
• add the "false" parameter to keep the slideshow from auto starting
• switch back to the visual editor, the "false" parameter does not work
• switch back to text editor and see the additional code that's been added ​https://screenshots.firefox.com/Qj0y2yZpdbtFVslK/wordpress.com

The "false" parameter will stick if you do not leave the text editor and save the post and publish it from that view.

[othellobloke]

Replying to azaozz:

Hi @othellobloke, from the support forum thread seems this was an issue with a specific theme that has been resolved. Closing as invalid, feel free to reopen with more info and steps to reproduce on a clean install.

It's not a theme issue. It's a TinyMCE issue.

[azaozz]

Think I got to the bottom of this. It is a problem with shortcodes (again!) and keeping the scroll position. When switching Text => Visual the keep-position marker span gets stuck inside of the shortcode preview/placeholder. Then switching back to Text restores it as part of the shortcode. Having the cursor in that span or in the shortcode repeats this.

Replying to othellobloke:

Right, it is a problem with shortcodes, previews and keeping scroll position.

Patch coming up.

[azaozz]

In 42574:

Editor: when keeping the scroll position treat all shortcodes as preview-able. Otherwise in some cases the marker span can get stuck inside the shortcode and break the markup.

Props azaozz, lizkarkoski and othellobloke for testing.
Fixes #42908 for trunk.

[azaozz]

Reopen for 4.9.3.

[azaozz]

In 42575:

Merged revision 42574 from trunk:

Editor: when keeping the scroll position treat all shortcodes as preview-able. Otherwise in some cases the marker span can get stuck inside the shortcode and break the markup.

Props azaozz, lizkarkoski and othellobloke for testing.
Fixes #42908 for 4.9.3.

[asadmalikdotpk]

Have the same issue using WordPress latest Version with Visual composer:
<span style="display: inline-block; width: 0px; overflow: hidden; line-height: 0;" data-mce-type="bookmark" class="mce_SELRES_start"></span>

[eclare]

Same issue here on a WordPres.org install (latest version), using a self-built theme and no Visual Composer. I do run Windows Vista with an outdated Chrome though, but I doubt that's an issue here.

[archon810]

Still a problem that exists in the latest WP 5.3.2.