Changeset 1042 for branches/new-dialogs/modules/Opera

Timestamp:

10/01/08 15:52:00 (4 years ago)

Author:

douglas

Message:

FIXED Ticket #1276 Update selectNodeContents to collapse in one direction or another, not to a specific location. Update docs to reflect this

Files:

• branches/new-dialogs/modules/Opera/Opera.js (modified) (2 diffs)

branches/new-dialogs/modules/Opera/Opera.js

@@ -576 +576 @@
  *
  * @param node DomNode 
- * @param pos  Set to a numeric position inside the node to collapse the cursor here if possible. 
- */
- 
-Xinha.prototype.selectNodeContents = function(node, pos)
+ * @param collapseToStart A boolean that, when supplied, says to collapse the selection. True collapses to the start, and false to the end.
+ */
+ 
+Xinha.prototype.selectNodeContents = function(node, collapseToStart)
 {
   this.focusEditor();
   this.forceRedraw();
   var range;
-  var collapsed = typeof pos == "undefined" ? true : false;
+  var collapsed = typeof collapseToStart == "undefined" ? true : false;
   var sel = this.getSelection();
   range = this._doc.createRange();
@@ -595 +595 @@
   {
     range.selectNodeContents(node);
-    //(collapsed) && range.collapse(pos);
   }
   sel.removeAllRanges();
   sel.addRange(range);
+  if (typeof collapseToStart != "undefined")
+  {
+    if (collapseToStart)
+    {
+      sel.collapse(range.startContainer, range.startOffset);
+    } else
+    {
+      sel.collapse(range.endContainer, range.endOffset);
+    }
+  }
 };