Remove another XXX
authorAryeh Gregor <AryehGregor+gitcommit@gmail.com>
Mon, 30 May 2011 14:31:34 -0600
changeset 203 152edd1e3cca
parent 202 f5a784f57311
child 204 9e4fccc76232
Remove another XXX

Should have been fixed by commit 947b0f87 "Attempt to fix selection
position bug better".
editcommands.html
source.html
--- a/editcommands.html	Mon May 30 14:28:57 2011 -0600
+++ b/editcommands.html	Mon May 30 14:31:34 2011 -0600
@@ -4105,10 +4105,6 @@
 <p class=XXX>Does not handle cases where block-extending the selection leaves
 it collapsed, like <code title="">&lt;p&gt;foo&lt;/p&gt;{}&lt;p&gt;bar&lt;/p&gt;</code>.
 
-<p class=XXX>The algorithm for preserving ranges doesn't work well here at all
-if the selection is collapsed.  The cursor can end up in the wrong place
-entirely.  Probably due to the wrap algorithm; that needs to be reexamined.
-
 <ol>
   <li>Let <var title="">items</var> be a list of all <code class=external data-anolis-spec=html title="the li element"><a href=http://www.whatwg.org/html/#the-li-element>li</a></code>s that are
   <a class=external data-anolis-spec=domrange href=http://html5.org/specs/dom-range.html#ancestor-container title="ancestor container">ancestor containers</a> of the <a href=#active-range>active range</a>'s <a class=external data-anolis-spec=domrange href=http://html5.org/specs/dom-range.html#concept-range-start title=concept-range-start>start</a>
--- a/source.html	Mon May 30 14:28:57 2011 -0600
+++ b/source.html	Mon May 30 14:31:34 2011 -0600
@@ -4131,10 +4131,6 @@
 <p class=XXX>Does not handle cases where block-extending the selection leaves
 it collapsed, like <code title>&lt;p>foo&lt;/p>{}&lt;p>bar&lt;/p></code>.
 
-<p class=XXX>The algorithm for preserving ranges doesn't work well here at all
-if the selection is collapsed.  The cursor can end up in the wrong place
-entirely.  Probably due to the wrap algorithm; that needs to be reexamined.
-
 <ol>
   <li>Let <var>items</var> be a list of all [[li]]s that are
   [[ancestorcontainers]] of the <span>active range</span>'s [[rangestart]]