Ticket #3603 (closed defect (bug): duplicate)

Opened 5 years ago

Last modified 10 months ago

end-of-comment markers converted to em-dash

Reported by: nickshanks Owned by:
Priority: normal Milestone:
Component: Formatting Version: 2.1
Severity: normal Keywords:
Cc:

Description

Enter the following in a new post:

string1 <!-- string2 <ol/> --> string3

You will get the output:

<p>string1 <!-- string2</p> <ol/> &#8211;> string3</p>

I don't know if other block-level elements suffer this same fate, but this is not even well-formed output!

Attachments

3603.diff Download (1014 bytes) - added by Nazgul 5 years ago.

Change History

Enter the following in a new post:

{{{string1 <!-- string2 <ol/> --> string3}}}

You will get the output:

{{{<p>string1 <!-- string2</p> <ol/> &#8211;> string3</p>}}}

I don't know if other block-level elements suffer this same fate, but this is not even well-formed output! }}}

trac is broken. i give up :P

  • Milestone changed from 2.2 to 2.1

Confirmed bug
ENV: 2.1-beta r44759

Seems to gooble the -- whenever there is an html tag in the comment.

Setting milestone as 2.1 as it seems better to disable em-dashing then have this behavior.

  • Milestone changed from 2.1 to 2.2

Nazgul5 years ago

  • Keywords has-patch added
  • Milestone changed from 2.2 to 2.3
  • Milestone changed from 2.3 to 2.4
  • Keywords needs-patch added; has-patch removed

Did some more testing. Although my patch fixes the problem it introduces others, so please ignore it.

Also closed #5073 as a duplicate of this bug.

  • Owner anonymous deleted
  • Component changed from General to Formatting

It is not possible to fix those things in wordpress. I suggest closing this as wontfix.

If you do not think so, please provide a patch that is actually fixing the issue and not breaking anything else.

  • Milestone changed from 2.9 to Future Release
  • Keywords needs-patch removed
  • Status changed from new to closed
  • Resolution set to duplicate

#8912 - that one seems more active

  • Milestone Future Release deleted
Note: See TracTickets for help on using tickets.