[wp-trac] [WordPress Trac] #31796: TinyMCE view - rendering twice

[WordPress Trac]

#31796: TinyMCE view - rendering twice
--------------------------+-------------------------
 Reporter:  mattheu       |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  4.2
Component:  TinyMCE       |     Version:  trunk
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:  javascript
--------------------------+-------------------------
Description changed by iseulde:

Old description:

> I have a query about this changeset:
> https://core.trac.wordpress.org/changeset/31832
>
> The following code was removed:
>

> {{{
> if ( selected ) {
>     removeView( selected );
> }
> }}}
>
> The commit is a revert of #31817 and #31819 - however I'm not sure why
> this code was removed because it wasn't added in either of these.
>
> This has broken the behaviour of my views - which are now rendered double
> if an existing view is updated.
>
> However this issue doesn't appear to be affecting the built-in previews
> such as audio or gallery so it may be that I am implementing things
> incorrectly!

New description:

 I have a query about this changeset:
 https://core.trac.wordpress.org/changeset/31832

 The following code was removed:


 {{{
 if ( selected ) {
     removeView( selected );
 }
 }}}

 The commit is a revert of [31817] and [31819] - however I'm not sure why
 this code was removed because it wasn't added in either of these.

 This has broken the behaviour of my views - which are now rendered double
 if an existing view is updated.

 However this issue doesn't appear to be affecting the built-in previews
 such as audio or gallery so it may be that I am implementing things
 incorrectly!

--

--
Ticket URL: <https://core.trac.wordpress.org/ticket/31796#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform