RESOLVED DUPLICATE of bug 7508 8811
REGRESSION: TinyMCE: Style toolbar buttons don't work 
https://bugs.webkit.org/show_bug.cgi?id=8811
Summary REGRESSION: TinyMCE: Style toolbar buttons don't work
Justin Garcia
Reported 2006-05-09 14:00:19 PDT
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.
Attachments
Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2006-06-04 16:40:21 PDT
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.
Darin Adler
Comment 2 2006-06-04 17:01:25 PDT
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.
Darin Adler
Comment 3 2006-06-05 08:28:42 PDT
Given my analysis, this is a duplicate of bug 7508. *** This bug has been marked as a duplicate of 7508 ***
Note You need to log in before you can comment on or make changes to this bug.