mirrors/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-04-21 03:55:24 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

LibWeb: Make select element use option's label, not text content

Browse source
This commit is contained in:
Psychpsyo 2024-11-16 22:14:42 +01:00 • committed by Sam Atkins
parent 8c4e4ec31b
commit 8f11dfc08a
Notes: github-actions[bot] 2025-01-04 11:45:06 +00:00
5 changed files with 28 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Libraries/LibWeb/HTML/HTMLOptionElement.cpp
View file

@ -114,6 +114,13 @@ String HTMLOptionElement::label() const
void HTMLOptionElement::set_label(String const& label)
{
MUST(set_attribute(HTML::AttributeNames::label, label));
// NOTE: This option's select element may need to show different contents now.
if (selected()) {
if (auto select_element = first_ancestor_of_type<HTMLSelectElement>()) {
select_element->update_inner_text_element({});
}
}
}
// https://html.spec.whatwg.org/multipage/form-elements.html#dom-option-text

14
Libraries/LibWeb/HTML/HTMLSelectElement.cpp
View file

@ -413,7 +413,7 @@ void HTMLSelectElement::show_the_picker_if_applicable()
for (auto const& child : opt_group_element.children_as_vector()) {
if (is<HTMLOptionElement>(*child)) {
auto& option_element = verify_cast<HTMLOptionElement>(*child);
option_group_items.append(SelectItemOption { id_counter++, option_element.selected(), option_element.disabled(), option_element, strip_newlines(option_element.text_content()), option_element.value() });
option_group_items.append(SelectItemOption { id_counter++, option_element.selected(), option_element.disabled(), option_element, strip_newlines(option_element.label()), option_element.value() });
}
}
m_select_items.append(SelectItemOptionGroup { opt_group_element.get_attribute(AttributeNames::label).value_or(String {}), option_group_items });
@ -421,7 +421,7 @@ void HTMLSelectElement::show_the_picker_if_applicable()
if (is<HTMLOptionElement>(*child)) {
auto& option_element = verify_cast<HTMLOptionElement>(*child);
m_select_items.append(SelectItemOption { id_counter++, option_element.selected(), option_element.disabled(), option_element, strip_newlines(option_element.text_content()), option_element.value() });
m_select_items.append(SelectItemOption { id_counter++, option_element.selected(), option_element.disabled(), option_element, strip_newlines(option_element.label()), option_element.value() });
}
if (is<HTMLHRElement>(*child))
@ -567,15 +567,21 @@ void HTMLSelectElement::create_shadow_tree_if_needed()
update_inner_text_element();
}
void HTMLSelectElement::update_inner_text_element(Badge<HTMLOptionElement>)
{
update_inner_text_element();
}
// FIXME: This needs to be called any time the selected option's children are modified.
void HTMLSelectElement::update_inner_text_element()
{
if (!m_inner_text_element)
return;
// Update inner text element to text content of selected option
// Update inner text element to the label of the selected option
for (auto const& option_element : list_of_options()) {
if (option_element->selected()) {
m_inner_text_element->set_text_content(strip_newlines(option_element->text_content()));
m_inner_text_element->set_text_content(strip_newlines(option_element->label()));
return;
}
}

2
Libraries/LibWeb/HTML/HTMLSelectElement.h
View file

@ -96,6 +96,8 @@ public:
void update_selectedness();
void update_inner_text_element(Badge<HTMLOptionElement>);
private:
HTMLSelectElement(DOM::Document&, DOM::QualifiedName);

4
Tests/LibWeb/Ref/expected/select-option-use-label.html Normal file
View file

@ -0,0 +1,4 @@
<!doctype html>
<select>
<option>label</option>
</select>

5
Tests/LibWeb/Ref/input/select-option-use-label.html Normal file
View file

@ -0,0 +1,5 @@
<!doctype html>
<link rel="match" href="../expected/select-option-use-label.html" />
<select>
<option label="label"></option>
</select>