WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#4241 closed defect (bug) (fixed)

Safari team says webkit build 420 is 100% TinyMCE compat

Reported by: andy Owned by: rob1n
Milestone: 2.2 Priority: normal
Severity: normal Version:
Component: TinyMCE Keywords: has-patch commit
Focuses: Cc:

Description

Attaching a patch to allow AppleWebKit/420 and higher to use RTE.

Attachments (1)

webkit420.diff (910 bytes) - added by andy 7 years ago.

Download all attachments as: .zip

Change History (8)

andy7 years ago

comment:1 andy7 years ago

By the way, I tested this by reducing the threshold to 419, which matches my build, and reloading the editor. It works just fine.

comment:2 rob1n7 years ago

  • Keywords has-patch commit added
  • Owner changed from anonymous to rob1n
  • Status changed from new to assigned

comment:3 rob1n7 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [5425]) Let WebKit 420 or above get in. And clean up that overgrown ternary. Props Andy. fixes #4241

comment:4 foolswisdom7 years ago

  • Milestone changed from 2.3 to 2.2
  • Resolution fixed deleted
  • Status changed from closed to reopened

Re-opening for consideration for 2.2 . Justification:

  • Low risk, non-intrusive for other users. Trunk and 2.2 currently have the same version of TinyMCE
  • Mac friendly features result in good blog/press coverage, encouraging adoption

comment:5 ryan7 years ago

(In [5433]) Add missing preg delimiter. see #4241

comment:6 rob1n7 years ago

I'm fine with this going into 2.2.

comment:7 rob1n7 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [5443]) Let WebKit 420 and above in for TinyMCE. fixes #4241 for 2.2

Note: See TracTickets for help on using tickets.