Demote XXX to TODO V2
authorAryeh Gregor <AryehGregor+gitcommit@gmail.com>
Mon, 11 Jul 2011 13:58:51 -0600
changeset 395 5ab04f33ebba
parent 394 373d18bf5a0e
child 396 e4328ccf371a
Demote XXX to TODO V2
editcommands.html
source.html
--- a/editcommands.html	Mon Jul 11 13:53:21 2011 -0600
+++ b/editcommands.html	Mon Jul 11 13:58:51 2011 -0600
@@ -3125,6 +3125,17 @@
 execCommand(), so it should be no big problem if we require that such nesting
 not occur.  (Existing browsers behave weirdly and inconsistently when
 confronted with this kind of nesting.)
+
+The reason we need this is that otherwise it gets very confusing to figure out
+what happens in cases like trying to outdent
+  <ol><li>[foo<ol><li>bar]</ol>baz</ol>
+If we first normalize, then the natural answer is something like
+  <p>[foo<ol><li>bar]<li>baz</ol>
+but if we don't, we'd have to special-case in the toggle lists and outdent
+algorithms.  This might be worthwhile, but it's not at all clear, and what I
+have works okay, so I'll stick with it for now.
+
+TODO V2: Investigate fixing this.
 -->
 
 <ol>
@@ -3288,9 +3299,6 @@
   <var title="">start offset</var> to the <a class=external data-anolis-spec=domrange href=http://html5.org/specs/dom-range.html#concept-indexof title=concept-indexof>index</a> of the last such <code class=external data-anolis-spec=html title="the li element"><a href=http://www.whatwg.org/html/#the-li-element>li</a></code> in
   <a class=external data-anolis-spec=domcore href=http://dvcs.w3.org/hg/domcore/raw-file/tip/Overview.html#tree-order>tree order</a>, and set <var title="">start node</var> to that <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 <a class=external data-anolis-spec=domcore href=http://dvcs.w3.org/hg/domcore/raw-file/tip/Overview.html#concept-tree-parent title=concept-tree-parent>parent</a>.
 
-  <p class=XXX>This plus the normalizing sublists stuff seems very messy.  Can
-  we do better?
-
   <li>Repeat the following steps:
 
   <ol>
--- a/source.html	Mon Jul 11 13:53:21 2011 -0600
+++ b/source.html	Mon Jul 11 13:58:51 2011 -0600
@@ -3101,6 +3101,17 @@
 execCommand(), so it should be no big problem if we require that such nesting
 not occur.  (Existing browsers behave weirdly and inconsistently when
 confronted with this kind of nesting.)
+
+The reason we need this is that otherwise it gets very confusing to figure out
+what happens in cases like trying to outdent
+  <ol><li>[foo<ol><li>bar]</ol>baz</ol>
+If we first normalize, then the natural answer is something like
+  <p>[foo<ol><li>bar]<li>baz</ol>
+but if we don't, we'd have to special-case in the toggle lists and outdent
+algorithms.  This might be worthwhile, but it's not at all clear, and what I
+have works okay, so I'll stick with it for now.
+
+TODO V2: Investigate fixing this.
 -->
 
 <ol>
@@ -3266,9 +3277,6 @@
   <var>start offset</var> to the [[index]] of the last such [[li]] in
   [[treeorder]], and set <var>start node</var> to that [[li]]'s [[parent]].
 
-  <p class=XXX>This plus the normalizing sublists stuff seems very messy.  Can
-  we do better?
-
   <li>Repeat the following steps:
 
   <ol>