Summary: | TinyMCE: Major regression in TinyMCE's Safari-specific selection code | ||
---|---|---|---|
Product: | WebKit | Reporter: | Justin Garcia <justin.garcia> |
Component: | HTML Editing | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | ian, mozillabug321 |
Priority: | P2 | ||
Version: | 420+ | ||
Hardware: | Mac | ||
OS: | OS X 10.4 | ||
URL: | http://tinymce.moxiecode.com/example.php?example=true | ||
Bug Depends on: | |||
Bug Blocks: | 6627 |
Description
Justin Garcia
2006-02-27 21:47:43 PST
Filed: https://sourceforge.net/tracker/index.php?func=detail&aid=1440124&group_id=103281&atid=635682 *** Bug 8811 has been marked as a duplicate of this bug. *** 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. 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. TinyMCE says it is fixed in their CVS - someone should verify. |