From a66c13ecf33cfeef679d1d019c8a98c9cdca99de Mon Sep 17 00:00:00 2001
From: Navid EMAD <navid.emad@yespark.fr>
Date: Sat, 13 Jun 2026 15:12:17 +0200
Subject: [PATCH] webapi: implement HTMLSelectElement.type and
 HTMLOptionElement.label

Both IDL attributes were unimplemented and evaluated to `undefined`.
`select.type` reflects the element's mode: "select-multiple" when the
`multiple` attribute is present, "select-one" otherwise (derived from the
existing `getMultiple()`). `option.label` returns the `label` content
attribute when present and non-empty, otherwise the value of the `text`
IDL attribute. Both match the HTML Living Standard.

Select-enhancement libraries branch on `select.type` to choose single- vs
multi-select code paths; reading `undefined` there made them take a dead
path and throw during initialization.

Closes #2738
---
 src/browser/tests/element/html/option.html | 17 +++++++++++++++++
 src/browser/tests/element/html/select.html | 14 ++++++++++++++
 src/browser/webapi/element/html/Option.zig | 16 ++++++++++++++++
 src/browser/webapi/element/html/Select.zig |  8 ++++++++
 4 files changed, 55 insertions(+)

diff --git a/src/browser/tests/element/html/option.html b/src/browser/tests/element/html/option.html
index 459833702..ec99e8991 100644
--- a/src/browser/tests/element/html/option.html
+++ b/src/browser/tests/element/html/option.html
@@ -29,6 +29,23 @@
   testing.expectEqual('Text 3', $('#opt3').text)
 </script>
 
+<option id="opt_labelled" label="Explicit Label">Inner Text</option>
+
+<script id="label">
+  // https://html.spec.whatwg.org/multipage/form-elements.html#dom-option-label
+  // No label attribute: falls back to the text.
+  testing.expectEqual('Text 1', $('#opt1').label)
+  // label attribute present and non-empty: returned as-is.
+  testing.expectEqual('Explicit Label', $('#opt_labelled').label)
+  // Setting reflects to the label content attribute.
+  $('#opt1').label = 'Set Label'
+  testing.expectEqual('Set Label', $('#opt1').label)
+  testing.expectEqual('Set Label', $('#opt1').getAttribute('label'))
+  // An empty label attribute falls back to the text, per spec.
+  $('#opt1').setAttribute('label', '')
+  testing.expectEqual('Text 1', $('#opt1').label)
+</script>
+
 <script id="text_set">
   $('#opt1').text = 'New Text 1'
   testing.expectEqual('New Text 1', $('#opt1').text)
diff --git a/src/browser/tests/element/html/select.html b/src/browser/tests/element/html/select.html
index 4c4e6ccde..71e9616ed 100644
--- a/src/browser/tests/element/html/select.html
+++ b/src/browser/tests/element/html/select.html
@@ -142,6 +142,20 @@
   }
 </script>
 
+<script id="type_reflects_multiple">
+  {
+    // https://html.spec.whatwg.org/multipage/form-elements.html#dom-select-type
+    testing.expectEqual('select-one', $('#select1').type)
+    testing.expectEqual('select-multiple', $('#select_multi').type)
+
+    // type tracks the multiple attribute, it is not stored independently
+    const sel = document.createElement('select')
+    testing.expectEqual('select-one', sel.type)
+    sel.multiple = true
+    testing.expectEqual('select-multiple', sel.type)
+  }
+</script>
+
 <script id="multiple_setValue">
   {
     const sel = $('#select_multi')
diff --git a/src/browser/webapi/element/html/Option.zig b/src/browser/webapi/element/html/Option.zig
index 8b4a22ba0..3a5b0d670 100644
--- a/src/browser/webapi/element/html/Option.zig
+++ b/src/browser/webapi/element/html/Option.zig
@@ -108,6 +108,21 @@ pub fn setName(self: *Option, name: []const u8, frame: *Frame) !void {
     try self.asElement().setAttributeSafe(comptime .wrap("name"), .wrap(name), frame);
 }
 
+// https://html.spec.whatwg.org/multipage/form-elements.html#dom-option-label
+// On getting, return the `label` content attribute if present and non-empty,
+// otherwise the value of the `text` IDL attribute. On setting, reflect to the
+// `label` content attribute.
+pub fn getLabel(self: *const Option, frame: *Frame) []const u8 {
+    if (self.asConstElement().getAttributeSafe(comptime .wrap("label"))) |label| {
+        if (label.len != 0) return label;
+    }
+    return self.getText(frame);
+}
+
+pub fn setLabel(self: *Option, label: []const u8, frame: *Frame) !void {
+    try self.asElement().setAttributeSafe(comptime .wrap("label"), .wrap(label), frame);
+}
+
 pub const JsApi = struct {
     pub const bridge = js.Bridge(Option);
 
@@ -119,6 +134,7 @@ pub const JsApi = struct {
 
     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 });
     pub const selected = bridge.accessor(Option.getSelected, Option.setSelected, .{});
     pub const defaultSelected = bridge.accessor(Option.getDefaultSelected, null, .{});
     pub const disabled = bridge.accessor(Option.getDisabled, Option.setDisabled, .{ .ce_reactions = true });
diff --git a/src/browser/webapi/element/html/Select.zig b/src/browser/webapi/element/html/Select.zig
index ddcb1791e..262a4d1c9 100644
--- a/src/browser/webapi/element/html/Select.zig
+++ b/src/browser/webapi/element/html/Select.zig
@@ -129,6 +129,13 @@ pub fn getMultiple(self: *const Select) bool {
     return self.asConstElement().getAttributeSafe(comptime .wrap("multiple")) != null;
 }
 
+// https://html.spec.whatwg.org/multipage/form-elements.html#dom-select-type
+// The `type` IDL attribute reflects the element's mode: "select-multiple" when
+// the `multiple` attribute is present, "select-one" otherwise.
+pub fn getType(self: *const Select) []const u8 {
+    return if (self.getMultiple()) "select-multiple" else "select-one";
+}
+
 pub fn setMultiple(self: *Select, multiple: bool, frame: *Frame) !void {
     if (multiple) {
         try self.asElement().setAttributeSafe(comptime .wrap("multiple"), .wrap(""), frame);
@@ -318,6 +325,7 @@ pub const JsApi = struct {
     };
 
     pub const value = bridge.accessor(Select.getValue, Select.setValue, .{});
+    pub const @"type" = bridge.accessor(Select.getType, null, .{});
     pub const selectedIndex = bridge.accessor(Select.getSelectedIndex, Select.setSelectedIndex, .{});
     pub const multiple = bridge.accessor(Select.getMultiple, Select.setMultiple, .{ .ce_reactions = true });
     pub const disabled = bridge.accessor(Select.getDisabled, Select.setDisabled, .{ .ce_reactions = true });