Ticket #4241 (closed defect (bug): fixed)

Opened 5 years ago

Last modified 5 years ago

Safari team says webkit build 420 is 100% TinyMCE compat

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

Description

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

Attachments

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

Change History

andy — 5 years ago

comment:1   andy — 5 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   rob1n — 5 years ago

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

comment:3   rob1n — 5 years ago

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

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

comment:4   foolswisdom — 5 years ago

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

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   ryan — 5 years ago

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

comment:6   rob1n — 5 years ago

I'm fine with this going into 2.2.

comment:7   rob1n — 5 years ago

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

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

Note: See TracTickets for help on using tickets.