[wp-trac] [WordPress Trac] #28940: TinyMCE strips out empty <i> tags, breaking font awesome

[WordPress Trac]

#28940: TinyMCE strips out empty <i> tags, breaking font awesome
-------------------------------------------------+-------------------------
 Reporter:  DonSailieri                          |       Owner:
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  Future
Component:  TinyMCE                              |  Release
 Severity:  normal                               |     Version:  3.9.1
 Keywords:  has-patch needs-testing dev-         |  Resolution:
  feedback                                       |     Focuses:
-------------------------------------------------+-------------------------

Comment (by ruud@…):

 See #29171

 Hi Azaozz,

 I agree with you on the removing of empty spans. However your patch (and
 the 29171.patch) seems to pad only those tags with an extra attribute and
 is not padding real empty span, which get removed subsequently in the
 process I presume?

 I disagree somewhat with you about whether or not this should be plugin
 material. Yes new features (like supporting schema) could very well be
 plugin material, but these simple use cases (with empty i and span tags)
 were working before in older WP versions, so stripping these out and
 providing plugins to compensate for that, is still a loss of functionality
 from an end-user perspective.

 Is it an idea to combine both patches and use these in version 4.0?

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