8811 2006-05-09 14:00:19 -0700 REGRESSION: TinyMCE: Style toolbar buttons don't work 2006-06-05 08:28:42 -0700 1 1 1 Unclassified WebKit HTML Editing 420+ Mac OS X 10.4 RESOLVED DUPLICATE 7508 http://tinymce.moxiecode.com/example.php?example=true Regression P1 Normal --- 6627 1 justin.garcia webkit-unassigned oldest_to_newest 41634 0 justin.garcia 2006-05-09 14:00:19 -0700 When you click on Bold, Italic, Underline or Strikethrough, the current typing style doesn't change. This used to work. But, we went back into the nightly build history and couldn't find a build where it worked. This probably means that the regression is in TinyMCE sources. 44711 1 darin 2006-06-04 16:40:21 -0700 The immediate cause for the fact that the bold button doesn't work is that the page calls window.selection.setBaseAndExtent to change the selection, and all 4 parameters are "undefined". This results in clearing the selection. I don't know what previous problem causes it to try to set the selection to undefined, undefined, undefined, undefined. There are two calls to setBaseAndExtent in the TinyMCE JavaScript, and the one that could pass those values looks like the one in moveToBookmark. 44720 2 darin 2006-06-04 17:01:25 -0700 Yes, the code that's going awry is some Safari-specific code that's attempting to work around the fact that the selection gets cleared in older versions of Safari. If you change the user agent string to Firefox, the toolbar buttons work just fine. There's something called lastSafariSelection. 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. 44770 3 darin 2006-06-05 08:28:42 -0700 Given my analysis, this is a duplicate of bug 7508. *** This bug has been marked as a duplicate of 7508 ***