From de53ca9205f01ec54d38afe4f74c313584540903 Mon Sep 17 00:00:00 2001 From: David Greenspan Date: Tue, 7 Aug 2012 19:40:48 -0700 Subject: [PATCH] more XXX TODOs --- packages/spark/spark.js | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/packages/spark/spark.js b/packages/spark/spark.js index f84246862d..6cd6dfe8bd 100644 --- a/packages/spark/spark.js +++ b/packages/spark/spark.js @@ -18,9 +18,17 @@ // path, or if you have multiple preserve nodes in a landmark with the // same selector and label -// getCurrentLandmark always searches up +// XXX event handling passes args (event, landmark) to handler, +// where `landmark` is the immediately enclosing landmark of the +// attachEvents range. -// getCurrentLandmark creates a dummy renderer if there isn't one +// XXX createLandmark takes an htmlFunc, which takes the landmark +// as its argument. There is no getCurrentLandmark. If there's +// no renderer, we createLandmark doesn't create an annotation, +// but it does instantiate a new Landmark for the sake of the +// htmlFunc (and create/destroy it inline). + +// XXX delete getEnclosingLandmark (unless used privately by tests) (function() {