7275 2006-02-15 08:40:34 -0800 window.getSelection() doesn't work in textareas 2011-06-01 04:46:48 -0700 1 1 1 Unclassified WebKit DOM 420+ Mac OS X 10.4 NEW P2 Major --- 1 stamm webkit-unassigned ap dgatwood gavin.sharp ian justin.garcia rniwa xan.lopez oldest_to_newest 32710 0 stamm 2006-02-15 08:40:34 -0800 window.getSelection() returns null when the selection is in a text area. This behavior makes it very difficult when there are two text areas on the page. Since the selection for a text area remains when you leave it, you have no idea which one of the text areas is the current (or last one) selected. In the latest nightly build, when I try the page below, if you select the "Random Text" and then click dump range, you will get a real range object. If you select the "My text" inside the text area, you get null for a range object. You can fake it using getSelectionRange, but that doesn't work. 1. Select "text" inside the text area 2. Select "random" 3. Click "Copy TextArea Sel". This contains the last selection. Even though random was selected. If I would add another text area, I would have no idea what the user has selected <html> <head> <meta http-equiv="Content-type" content="text/html" /> <title>Simple Selection Test</title> <script> function dumpO(o) { var msg = ""; var i = 0; for (var f in o) { try { msg += f + " = " + o[f] + "\n"; } catch (e) {} if (i++ == 10) { i=0; alert(msg); msg = ""; } } if (msg) alert(msg); } function getSelectionRange(o) { var start = o.selectionStart; var end = o.selectionEnd; return o.value.substring(start,end); } </script> </head> <body> <button onclick="document.getElementById('r').innerHTML = window.getSelection();"><u>Copy Window Sel</u></button><br> <button onclick="document.getElementById('r').innerHTML = getSelectionRange(document.getElementById('text'));"><u>Copy TextArea Sel</u></button><br> <button onclick="dumpO(window.getSelection());"><u>Dump Selection</u></button><br> <button onclick="dumpO(window.getSelection().getRangeAt(0));"><u>Dump Range</u></button><br> <button onclick="document.getElementById('r').innerHTML = window.getSelection().getRangeAt(0);"><u>Copy Range</u></button><br> <button onclick="dumpO(document.getElementById('text'));"><u>Dump TextArea</u></button><br> <button onclick="dumpO(window);"><u>Dump Window</u></button><br> <textarea id="text" name="text" rows="20" cols="80"/> My text</textarea> <div id="r">divMe</div> Random Text </body> </html> 32736 1 ap 2006-02-15 12:17:40 -0800 Confirming as a difference between Firefox and WebKit, but I am not sure what the correct approach would be. In Firefox, window.getSelection() returns a collapsed range positioned before the TEXTAREA (HTMLBodyElement, offset 22 in your example). This doesn't look right to me - that is an entirely different position, and the insertion point can easily be moved there in contenteditable mode. TEXTAREA has a child text node - perhaps it would be more appropriate to make window.getSelection() return the actual range within this text node? BTW, you can probably use onfocus/onblur handlers to track the last focused textarea. 47855 2 stamm 2006-06-30 12:11:16 -0700 > BTW, you can probably use onfocus/onblur handlers to track the last focused textarea. This didn't work with 418 because onblur didn't get called appropriately when you click on a button 9096 3 dgatwood 2007-05-25 16:21:23 -0700 This fails similarly for anything set to contentEditable, too. I've been smashing my head against a wall over this bug off and on for months. 65626 4 ap 2007-12-29 13:26:35 -0800 (In reply to comment #3) > This fails similarly for anything set to contentEditable, too. Could you please provide a test case (preferably, in a new bug, to keep this one focused on textareas)? This seems to work correctly to me. 77735 5 20565 eric 2008-04-15 15:16:02 -0700 Created attachment 20565 test case (same as is pasted as text in comments below) 413297 6 xan.lopez 2011-06-01 04:46:48 -0700 FWIW, I recently stumbled upon this bug and I think nowadays we do what comment #1 (from 2006!) says Firefox used to do back then. The thing that makes this fail is DOMSelection.cpp:getRangeAt, which has a chunk like: if (Node* shadowAncestor = selectionShadowAncestor(m_frame)) { ContainerNode* container = shadowAncestor->parentNodeGuaranteedHostFree(); int offset = shadowAncestor->nodeIndex(); return Range::create(shadowAncestor->document(), container, offset, container, offset); } triggered inside text areas (and that makes us return a 0-0 bogus range). If I just *remove* that code, my simple tests seem to work OK, but I'm sure that breaks something else somewhere. Opinions? 20565 2008-04-15 15:16:02 -0700 2008-04-15 15:16:02 -0700 test case (same as is pasted as text in comments below) test.html text/html 1410 eric PGh0bWw+CjxoZWFkPgo8bWV0YSBodHRwLWVxdWl2PSJDb250ZW50LXR5cGUiIGNvbnRlbnQ9InRl eHQvaHRtbCIgLz4KPHRpdGxlPlNpbXBsZSBTZWxlY3Rpb24gVGVzdDwvdGl0bGU+CjxzY3JpcHQ+ CmZ1bmN0aW9uIGR1bXBPKG8pIHsKICB2YXIgbXNnID0gIiI7CiAgdmFyIGkgPSAwOwogIGZvciAo dmFyIGYgaW4gbykgewogICAgdHJ5IHsKICAgICAgbXNnICs9IGYgKyAiID0gIiArIG9bZl0gKyAi XG4iOwogICAgfSBjYXRjaCAoZSkge30KICAgIGlmIChpKysgPT0gMTApIHsKICAgICAgIGk9MDsK ICAgICAgIGFsZXJ0KG1zZyk7CiAgICAgICBtc2cgPSAiIjsKICAgIH0KICB9CiAgaWYgKG1zZykg YWxlcnQobXNnKTsKfQpmdW5jdGlvbiBnZXRTZWxlY3Rpb25SYW5nZShvKSB7CiAgdmFyIHN0YXJ0 ID0gby5zZWxlY3Rpb25TdGFydDsKICB2YXIgZW5kID0gby5zZWxlY3Rpb25FbmQ7CiAgcmV0dXJu IG8udmFsdWUuc3Vic3RyaW5nKHN0YXJ0LGVuZCk7Cn0KCjwvc2NyaXB0Pgo8L2hlYWQ+Cjxib2R5 Pgo8YnV0dG9uIG9uY2xpY2s9ImRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCdyJykuaW5uZXJIVE1M ID0KICAgICAgICAgIHdpbmRvdy5nZXRTZWxlY3Rpb24oKTsiPjx1PkNvcHkgV2luZG93IFNlbDwv dT48L2J1dHRvbj48YnI+CjxidXR0b24gb25jbGljaz0iZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQo J3InKS5pbm5lckhUTUwgPQogICAgICAgICAgICAgICAgIGdldFNlbGVjdGlvblJhbmdlKGRvY3Vt ZW50LmdldEVsZW1lbnRCeUlkKCd0ZXh0JykpOyI+PHU+Q29weQogICAgICAgICAgICAgICAgIFRl eHRBcmVhIFNlbDwvdT48L2J1dHRvbj48YnI+CjxidXR0b24gb25jbGljaz0iZHVtcE8od2luZG93 LmdldFNlbGVjdGlvbigpKTsiPjx1PkR1bXAKU2VsZWN0aW9uPC91PjwvYnV0dG9uPjxicj4KPGJ1 dHRvbiBvbmNsaWNrPSJkdW1wTyh3aW5kb3cuZ2V0U2VsZWN0aW9uKCkuZ2V0UmFuZ2VBdCgwKSk7 Ij48dT5EdW1wClJhbmdlPC91PjwvYnV0dG9uPjxicj4KPGJ1dHRvbiBvbmNsaWNrPSJkb2N1bWVu dC5nZXRFbGVtZW50QnlJZCgncicpLmlubmVySFRNTCA9CndpbmRvdy5nZXRTZWxlY3Rpb24oKS5n ZXRSYW5nZUF0KDApOyI+PHU+Q29weSBSYW5nZTwvdT48L2J1dHRvbj48YnI+CjxidXR0b24gb25j bGljaz0iZHVtcE8oZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ3RleHQnKSk7Ij48dT5EdW1wClRl eHRBcmVhPC91PjwvYnV0dG9uPjxicj4KPGJ1dHRvbiBvbmNsaWNrPSJkdW1wTyh3aW5kb3cpOyI+ PHU+RHVtcCBXaW5kb3c8L3U+PC9idXR0b24+PGJyPgo8dGV4dGFyZWEgaWQ9InRleHQiIG5hbWU9 InRleHQiIHJvd3M9IjIwIiBjb2xzPSI4MCIvPgpNeSB0ZXh0PC90ZXh0YXJlYT4KPGRpdiBpZD0i ciI+ZGl2TWU8L2Rpdj4KUmFuZG9tIFRleHQKPC9ib2R5Pgo8L2h0bWw+