Fix checkbox DOM sync

src/Components/Web.JS/src/Rendering/DomMerging/DomSync.ts

@@ -354,8 +354,10 @@ function ensureEditableValueSynchronized(destination: Element, value: any) {
   } else if (destination instanceof HTMLSelectElement && destination.selectedIndex !== value) {
     destination.selectedIndex = value as number;
   } else if (destination instanceof HTMLInputElement) {
-    if (destination.type === 'checkbox' && destination.checked !== value) {
-      destination.checked = value as boolean;
+    if (destination.type === 'checkbox') {
+      if (destination.checked !== value) {
+        destination.checked = value as boolean;
+      }
     } else if (destination.value !== value) {
       destination.value = value as string;
     }

src/Components/Web.JS/test/DomSync.test.ts

@@ -469,6 +469,22 @@ describe('DomSync', () => {
     expect(selectElem.selectedIndex).toBe(2);
   });
 
+  test('should handle checkboxes with value attribute', () => {
+    // Checkboxes require even more special-case handling because their 'value' attribute
+    // has to be handled as a regular attribute, and 'checked' must be handled similarly
+    // to 'value' on other inputs
+
+    const destination = makeExistingContent(`<input type='checkbox' value='first' checked />`);
+    const newContent = makeNewContent(`<input type='checkbox' value='second' checked />`);
+
+    const checkboxElem = destination.startExclusive.nextSibling as HTMLInputElement;
+
+    // Act/Assert
+    synchronizeDomContent(destination, newContent);
+    expect(checkboxElem.checked).toBeTruthy();
+    expect(checkboxElem.value).toBe('second');
+  });
+
   test('should treat doctype nodes as unchanged', () => {
     // Can't update a doctype after the document is created, nor is there a use case for doing so
     // We just have to skip them, as it would be an error to try removing or inserting them