[Webkit-unassigned] [Bug 13399] Support document.getSelection()

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=13399


justin.garcia at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Support                     |Support
                   |document.getSelection() and |document.getSelection()
                   |document.selection          |




------- Comment #2 from justin.garcia at apple.com  2008-02-21 16:34 PDT -------
I don't think we want to add support for document.selection.  This is an IE
specific extention and their selection object is too different than Safari's or
FF's.  In a few of the dupes of <rdar://problem/3138411> I see code like this:

<a
href="javascript:bm=document.selection?document.selection.createRange().text:document.getSelection();void(pmentry=window.open('http://jens.apple.com/pm/index.php?m=bookmarklet&p=new&title='+escape(document.title)+'&url='+escape(window.location.href)+'&body='+escape(bm),'pmentry','width=700,height=520,toolbar=0,status=1,location=0,scrollbars=1,menubar=1,resizable=1'))">pMachineBM</a>

If we added support for document.selection we'd fall into the IE specific code
path, which has us calling more IE specific extentions that neither Safari nor
FF supports (selection.createRange() for example returns a TextRange object,
not a DOM Range object, which has all sorts of IE specific extentions). 
Perhaps at a later time if we extend DOM Range to support more methods we can
implement document.selection.

I think that for now, supporting code that works in FF is good enough.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.