7508 2006-02-27 21:47:43 -0800 TinyMCE: Major regression in TinyMCE's Safari-specific selection code 2006-08-12 23:50:58 -0700 1 1 1 Unclassified WebKit HTML Editing 420+ Mac OS X 10.4 RESOLVED FIXED http://tinymce.moxiecode.com/example.php?example=true P2 Normal --- 6627 1 justin.garcia webkit-unassigned ian mozillabug321 oldest_to_newest 34262 0 justin.garcia 2006-02-27 21:47:43 -0800 In the TinyMCE 2.0.3 update, a change was made that broke editing operations to selected text when performed via a toolbar button. Goto: http://tinymce.moxiecode.com/example.php?example=true Select some text, click the Bold button. The selection is blown away and nothing becomes bold. It sounds like some refactoring was done to the selection code between 2.0.2 and 2.0.3 by one of the TinyMCE contributor, which is probably where the regression was introduced. I'm filing a bug against TinyMCE now and I'll attach a link to that bug shortly. 34263 1 justin.garcia 2006-02-27 22:01:05 -0800 Filed: https://sourceforge.net/tracker/index.php?func=detail&aid=1440124&group_id=103281&atid=635682 44771 2 darin 2006-06-05 08:28:42 -0700 *** Bug 8811 has been marked as a duplicate of this bug. *** 44772 3 darin 2006-06-05 08:29:17 -0700 It looks like this section of the moveToBookmark function is the buggy Safari-specific TinyMCE code: if (tinyMCE.isSafari) { sel.setBaseAndExtent(bookmark.startContainer, bookmark.startOffset, bookmark.endContainer, bookmark.endOffset); return true; } That should be bookmark.rng.startContainer, bookmark.rng.startOffset, etc. A bookmark doesn't have properties with those names. 44773 4 darin 2006-06-05 08:55:55 -0700 I added some comments and a patch to the bug tracker for TinyMCE, but I don't know what it's going to take to get the attention of people working on the TinyMCE project. 52443 5 mjs 2006-08-12 22:20:58 -0700 TinyMCE says it is fixed in their CVS - someone should verify. 52458 6 mjs 2006-08-12 23:50:58 -0700 Verified with http://tinymce.moxiecode.com/test/examples/example_full.htm