[wp-trac] [WordPress Trac] #40951: New Text Widget - Switching Between Visual/Text Editor Strips Out Code

[WordPress Trac]

#40951: New Text Widget - Switching Between Visual/Text Editor Strips Out Code
-------------------------------------------------+-------------------------
 Reporter:  dwrippe                              |       Owner:
     Type:  defect (bug)                         |  westonruter
 Priority:  normal                               |      Status:  accepted
Component:  Widgets                              |   Milestone:  4.8.1
 Severity:  major                                |     Version:  4.8
 Keywords:  needs-testing has-unit-tests has-    |  Resolution:
  patch                                          |     Focuses:
-------------------------------------------------+-------------------------

Comment (by westonruter):

 Replying to [comment:127 Ov3rfly]:
 > A closer look shows another? problem in 4.8 code. Pre-existing html-
 comment gets wrapped in <p> when printed to front end, which leads to
 extra empty space in front end. Also extra linebreaks are added within the
 <p class="some-class>

 I presume you did not have the auto-paragraphing checkbox checked in this
 example? Please try editing a widget with those contents when the patch is
 applied to your install. I just tried, and I see it is presented in legacy
 mode. See also the [https://github.com/xwp/wordpress-develop/pull/235 PR].

 I may need to add the detection of HTML comments to also opt-in to legacy
 mode.

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