[wp-trac] [WordPress Trac] #37693: TinyMCE: backtics not handled correctly in paragraphs that already contain backticks

WordPress Trac noreply at wordpress.org
Fri Nov 4 09:11:18 UTC 2016


#37693: TinyMCE: backtics not handled correctly in paragraphs that already contain
backticks
--------------------------+-------------------------
 Reporter:  borekb        |       Owner:  iseulde
     Type:  defect (bug)  |      Status:  reviewing
 Priority:  normal        |   Milestone:  4.7
Component:  TinyMCE       |     Version:  4.3
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |     Focuses:  javascript
--------------------------+-------------------------

Comment (by iseulde):

 Replying to [comment:16 azaozz]:
 > 37693.9.patch looks good. 37693.10.patch adds some cleanup and changes
 it to test for a white space or start of line before the pattern, instead
 of looking for the first char of the pattern (as we discussed in #core-
 editor). So if the pattern is `*`, `test* test*` will not trigger it, and
 `test *test*` will.

 That looks good. The only case I have where I would expect it to trigger
 is something like `some*thing*`. I agree that we shouldn't allow it when
 there is whitespace after the starting pattern though. So maybe `some*
 thing*` should not trigger it, but `some *thing*` and `some*thing*`
 should.

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


More information about the wp-trac mailing list