Changeset 521

Timestamp:

08/09/08 19:58:01 (4 years ago)

Author:

s.lewis

Message:

Added code to plugin1 to test the SAPI.cacheSelection and SAPI.restoreSelection methods

Location:

branches/scott/0.6-a4/js/plugins

Files:

• plugin1/test.js (modified) (2 diffs)
• sapi/sapi.js (modified) (5 diffs)

branches/scott/0.6-a4/js/plugins/plugin1/test.js

@@ -18 +18 @@
             behaviors: {
                 mouseover: function(e) {
-                    $('#log').html('Plugin for<br /> WYMeditor Instance ' + e.data._index);
+                    // $('#log').html('Plugin for<br /> WYMeditor Instance ' + e.data.parent._index);
                 },
                 mouseout: function(e) {
-                    $('#log').html('');
+                    // $('#log').html('');
                 },
                 click: function(e) {
                     e.preventDefault();
+                    e.data.parent.sapi.cacheSelection();
                     $.get("html/paste.html", function(data) {
                         $.modal(data, {
@@ -30 +31 @@
                             overlay: 75,
                             onShow: function(dialog) {
-                                $(".wym_dialog .wym_cancel").bind('click', function(e) {
+                                $(".wym_dialog .wym_cancel").bind('click', e.data, function(e) {
+                                    e.preventDefault();
                                     $.modal.close();
+                                    e.data.parent.sapi.restoreSelection();
                                 });
                                 $(".wym_dialog .wym_submit").bind('click', e.data, function(e) {
                                     e.preventDefault();
                                     var editor = e.data.parent;
+                                    editor.sapi.restoreSelection();
                                     var text = $("#paste_text").val();
                                     if ($.trim(text) != "")

branches/scott/0.6-a4/js/plugins/sapi/sapi.js

@@ -221 +221 @@
     };
     
+    /**
+    * IMPORTANT: Because the selection/range object is part of the DOM, it is owned by 
+    * the parent window of the document which contains it. Therefore, the selection 
+    * within an editor must be cached within the scope of the parent window of the 
+    * editor. This means that the selection object must be stored in the iframe's 
+    * window.
+    */
+    
     sapi.prototype.cacheSelection = function() {
-        var w = this.parent._iframe.contentWindow;
-        var d = this.parent._doc;
+        $('#log').html('Caching selection<br />');
+        var w = this.parent.docWindow;
+        var d = this.parent.doc;
         if (w.getSelection) {
             var selection = w.getSelection();
@@ -248 +257 @@
     };
     
-    sapi.prototype.restoreSelection = function(index) {
-        var w = this.parent._iframe.contentWindow;
-        var d = this.parent._doc;
+    sapi.prototype.restoreSelection = function() {
+        $('#log').html($('#log').html()+'Re-storing selection<br />');
+        var w = this.parent.docWindow;
+        var d = this.parent.doc;
         try {jQuery(d).focus();} catch(e) {}
         if (w._selection) {
@@ -269 +279 @@
     
     sapi.replaceSelectionText = function(text) {
-        var w = this.parent._iframe.contentWindow;
-        var d = this.parent._doc;
+        var w = this.parent.docWindow;
+        var d = this.parent.doc;
         try {jQuery(d).focus();} catch(e) {}
         if (!w.getSelection && 
@@ -425 +435 @@
         {
             $.extend(p, {
-                method: "compareEndPoints",
+                method:       "compareEndPoints",
                 StartToStart: "StartToStart",
-                StartToEnd: "StartToEnd",
-                EndToEnd: "EndToEnd",
-                EndToStart: "EndToStart"
+                StartToEnd:   "StartToEnd",
+                EndToEnd:     "EndToEnd",
+                EndToStart:   "EndToStart"
             });
         } 
@@ -435 +445 @@
         {
             $.extend(p, {
-                method: "compareBoundaryPoints",
+                method:       "compareBoundaryPoints",
                 StartToStart: 0,
-                StartToEnd: 1,
-                EndToEnd: 2,
-                EndToStart: 3
+                StartToEnd:   1,
+                EndToEnd:     2,
+                EndToStart:   3
             });
         }