From b09049cbf36598ed215cd92132b41931223b4f4b Mon Sep 17 00:00:00 2001
From: Karl Seguin <k@openmymind.io>
Date: Thu, 24 Sep 2026 10:06:15 +0800
Subject: [PATCH] webapi: add new Option() support

---
 src/browser/tests/element/html/option.html | 66 ++++++++++++++++++++++
 src/browser/webapi/element/html/Option.zig | 22 ++++++++
 2 files changed, 88 insertions(+)

diff --git a/src/browser/tests/element/html/option.html b/src/browser/tests/element/html/option.html
index 0494f6b4a..e35cc306e 100644
--- a/src/browser/tests/element/html/option.html
+++ b/src/browser/tests/element/html/option.html
@@ -191,3 +191,69 @@
   $('#opt4').disabled = false
   testing.expectEqual(false, $('#opt4').disabled)
 </script>
+
+<script id="constructor">
+  {
+    const opt = new Option()
+    testing.expectEqual('OPTION', opt.tagName)
+    testing.expectEqual(true, opt instanceof HTMLOptionElement)
+    testing.expectEqual(0, opt.childNodes.length)
+    testing.expectEqual(false, opt.hasAttribute('value'))
+    testing.expectEqual(false, opt.defaultSelected)
+    testing.expectEqual(false, opt.selected)
+  }
+
+  {
+    const opt = new Option('Label', 'v1')
+    testing.expectEqual('Label', opt.text)
+    testing.expectEqual(1, opt.childNodes.length)
+    testing.expectEqual('v1', opt.getAttribute('value'))
+    testing.expectEqual('v1', opt.value)
+  }
+
+  {
+    // An empty text appends no Text node; null is stringified.
+    testing.expectEqual(0, new Option('').childNodes.length)
+    testing.expectEqual('null', new Option(null).text)
+    testing.expectEqual('', new Option('x', '').value)
+
+    const undef = new Option(undefined, undefined)
+    testing.expectEqual(0, undef.childNodes.length)
+    testing.expectEqual(false, undef.hasAttribute('value'))
+
+    const falsy = new Option(false, false)
+    testing.expectEqual('false', falsy.text)
+    testing.expectEqual('false', falsy.getAttribute('value'))
+  }
+
+  {
+    // WebIDL boolean conversion
+    const falsy = new Option('text', 'value', 0, '')
+    testing.expectEqual(false, falsy.hasAttribute('selected'))
+    testing.expectEqual(false, falsy.selected)
+
+    const truthy = new Option('text', 'value', {}, 1)
+    testing.expectEqual(true, truthy.hasAttribute('selected'))
+    testing.expectEqual(true, truthy.selected)
+  }
+
+  {
+    // defaultSelected sets the attribute, selectedness comes from `selected` only.
+    const opt = new Option('a', 'b', true)
+    testing.expectEqual(true, opt.defaultSelected)
+    testing.expectEqual('', opt.getAttribute('selected'))
+    testing.expectEqual(false, opt.selected)
+
+    const opt2 = new Option('a', 'b', false, true)
+    testing.expectEqual(false, opt2.defaultSelected)
+    testing.expectEqual(true, opt2.selected)
+  }
+
+  {
+    const select = document.createElement('select')
+    select.add(new Option('One', '1'))
+    select.add(new Option('Two', '2', false, true))
+    testing.expectEqual('2', select.value)
+    testing.expectEqual(1, select.selectedIndex)
+  }
+</script>
diff --git a/src/browser/webapi/element/html/Option.zig b/src/browser/webapi/element/html/Option.zig
index 42a5afce0..fea38fd64 100644
--- a/src/browser/webapi/element/html/Option.zig
+++ b/src/browser/webapi/element/html/Option.zig
@@ -39,6 +39,26 @@ _value: ?[]const u8 = null,
 _selected: bool = false,
 _default_selected: bool = false,
 
+pub fn constructor(text_: ?js.NullableString, value_: ?js.NullableString, default_selected_: ?bool, selected_: ?bool, frame: *Frame) !*Option {
+    const node = try Frame.node_factory.createElementNS(frame.document, .html, "option", null);
+    const el = node.as(Element);
+
+    const text = if (text_) |t| t.value else "";
+    if (text.len > 0) {
+        _ = try node.appendChild(try frame.document.createTextNode(text), frame);
+    }
+    if (value_) |v| {
+        try el.setAttributeSafe(comptime .wrap("value"), .wrap(v.value), frame);
+    }
+    if (default_selected_ orelse false) {
+        try el.setAttributeSafe(comptime .wrap("selected"), comptime .wrap(""), frame);
+    }
+
+    const self = el.as(Option);
+    self._selected = selected_ orelse false;
+    return self;
+}
+
 pub fn asElement(self: *Option) *Element {
     return Factory.protoOf(self).asElement();
 }
@@ -135,12 +155,14 @@ pub const JsApi = struct {
 
     pub const Meta = struct {
         pub const name = "HTMLOptionElement";
+        pub const constructor_alias = "Option";
         pub const prototype_chain = bridge.prototypeChain();
         pub var class_id: bridge.ClassId = undefined;
     };
 
     const reflect = Element.Reflect(Option);
 
+    pub const constructor = bridge.constructor(Option.constructor, .{});
     pub const value = bridge.accessor(Option.getValue, Option.setValue, .{ .ce_reactions = true });
     pub const text = bridge.accessor(Option.getText, Option.setText, .{ .ce_reactions = true });
     pub const label = bridge.accessor(Option.getLabel, Option.setLabel, .{ .ce_reactions = true });