add comments

preactjs · Jul 23, 2024 · e864f10 · e864f10
1 parent cff1c1b
commit e864f10
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/diff/index.js b/src/diff/index.js
@@ -435,7 +435,7 @@ function diffElementNodes(
 
 		oldProps = oldVNode.props || EMPTY_OBJ;
 
-		// TODO: do we want to keep this in v11?
+		// TODO: do we want to keep this in v11? Bigger question right now
 		// If we are in a situation where we are not hydrating but are using
 		// existing DOM we should read the existing DOM attributes to diff them
 		if (!isHydrating && excessDomChildren != null) {

diff --git a/test/browser/render.test.js b/test/browser/render.test.js
@@ -1217,6 +1217,7 @@ describe('render()', () => {
 		}
 	});
 
+	// Would break if we remove the mutative render props removal
 	it('should remove attributes on pre-existing DOM', () => {
 		const div = document.createElement('div');
 		div.setAttribute('class', 'red');