13830 2011-08-18 21:01:15 +0000 "Delete the contents" has to be rewritten to delete the selection 2011-08-30 19:54:50 +0000 1 1 1 Unclassified WebAppsWG HISTORICAL - HTML Editing APIs unspecified All All RESOLVED FIXED P2 normal --- 1 ayg ayg mike public-webapps sideshowbarker+html-editing-api oldest_to_newest 55442 0 ayg 2011-08-18 21:01:15 +0000 Currently the "delete the contents" algorithm deletes any old range it's given. Every caller except delete/forwardDelete actually deletes the active range. The problem with deleting non-active ranges is it doesn't update the selection correctly. Example: calling delete on <ol><li>foo</li></ol>{}<br> produces <ol><li>foo</li></ol>{} in the current spec because the selection doesn't get moved. delete/forwardDelete can maybe be changed to change the selection before deleting. Or maybe there's some other fix that's better. (This is a minor bug in the current spec, not an issue implementers have to worry about.) 56046 1 ayg 2011-08-30 16:52:08 +0000 This is turning out to be a surprisingly big headache. Apparently my algorithms are so complicated and/or poorly documented that I can't understand them a month or two after writing them. I'll have to add more comments and/or refactor -- but for now, at least I have a good test suite! 56066 2 ayg 2011-08-30 19:54:50 +0000 http://aryeh.name/gitweb.cgi?p=editing;a=commitdiff;h=bc8218cc Filed bug 13973 on refactoring this so it's less of a nightmare.