Make WordPress Core

Opened 18 years ago

Closed 18 years ago

#4241 closed defect (bug) (fixed)

Safari team says webkit build 420 is 100% TinyMCE compat

Reported by: andy's profile andy Owned by: rob1n's profile 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 18 years ago.

Download all attachments as: .zip

Change History (8)

@andy
18 years ago

#1 @andy
18 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.

#2 @rob1n
18 years ago

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

#3 @rob1n
18 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

#4 @foolswisdom
18 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

#5 @ryan
18 years ago

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

#6 @rob1n
18 years ago

I'm fine with this going into 2.2.

#7 @rob1n
18 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.