minor comment tweaks

This commit is contained in:
David Greenspan
2012-08-21 02:22:53 -04:00
parent f70aa390bb
commit a326aa367f
2 changed files with 5 additions and 4 deletions

View File

@@ -541,7 +541,7 @@ Spark.renderToRange = function (range, htmlFunc) {
var results = {};
// patch (using preservations)
// Patch! (using preservations)
range.operate(function (start, end) {
// XXX this will destroy all liveranges, including ones
// inside constant regions whose DOM nodes we are going
@@ -549,8 +549,8 @@ Spark.renderToRange = function (range, htmlFunc) {
Spark.finalize(start, end);
// Disable events during patching. If we don't do this, patching
// a checkbox in old IE will cause "change" events to fire on the
// partially-patched DOM. Since the liverange tree needn't be
// well-formed, delivering events mid-patch is never right.
// partially-patched DOM. Since the LiveRange tree needn't even
// be well-formed, delivering events mid-patch is never right.
Spark._eventsDisabled.withValue(true, function () {
Spark._patch(start.parentNode, frag, start.previousSibling,
end.nextSibling, preservations, results);

View File

@@ -2616,7 +2616,8 @@ Tinytest.add("spark - controls", function(test) {
// IE 7 is known to fire change events on all
// the radio buttons with checked=false, as if
// each button were deselected before selecting
// the new one.
// the new one. (Meteor doesn't normalize this
// behavior.)
// However, browsers are consistent if we are
// getting a checked=true notification.
var btn = event.target;