This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 13973 - Refactor delete stuff
Summary: Refactor delete stuff
Status: RESOLVED FIXED
Alias: None
Product: WebAppsWG
Classification: Unclassified
Component: HISTORICAL - HTML Editing APIs (show other bugs)
Version: unspecified
Hardware: All All
: P2 normal
Target Milestone: ---
Assignee: Aryeh Gregor
QA Contact: HTML Editing APIs spec bugbot
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 13831 13976
  Show dependency tree / graph
 
Reported: 2011-08-30 19:53 UTC by Aryeh Gregor
Modified: 2011-11-08 17:32 UTC (History)
2 users (show)

See Also:

Attachments

Description Aryeh Gregor 2011-08-30 19:53:47 UTC 
Everything in the spec related to deletion (delete the selection, delete command, forwardDelete command) is just scary.  In the course of fixing bug 13830, I had to change a zillion things to fix a cascade of successive regression test failures that I didn't fully understand.  The whole thing needs a giant injection of sanity.  I probably want to think carefully about what exactly I'm doing at every step and refactor it to be more intuitive.

What evil have I wrought?  :(
Comment 1 Aryeh Gregor 2011-11-07 23:13:00 UTC 
First steps:

https://dvcs.w3.org/hg/editing/rev/420aa6a5998b
Comment 2 Aryeh Gregor 2011-11-08 17:32:47 UTC 
I don't see much more to change, offhand.  If I run into more headaches when doing delete stuff, I guess I'll do more refactoring then of whatever parts seem to be flaky.