Editor: Improve accessibility for Editor and Project Settings

- Add accessibility names to EditorSpinSlider, inspector scrollbar, and settings categories tree
- Propagate property labels to focusable controls in EditorProperty::set_label
- Eliminate duplicate tab stops by setting EditorProperty to FOCUS_NONE when it has focusable children
- Fix EditorSettingsPropertyWrapper to pass label and set proper focus mode
- Skip setting empty 'Property:' accessibility names

Author: ndarilek

editor/gui/editor_spin_slider.cpp

@@ -461,6 +461,32 @@ void EditorSpinSlider::_draw_spin_slider() {
 
 void EditorSpinSlider::_notification(int p_what) {
 	switch (p_what) {
+		case NOTIFICATION_ACCESSIBILITY_UPDATE: {
+			RID ae = get_accessibility_element();
+			ERR_FAIL_COND(ae.is_null());
+
+			DisplayServer::get_singleton()->accessibility_update_set_role(ae, DisplayServer::AccessibilityRole::ROLE_SPIN_BUTTON);
+			// Use accessibility_name if set, otherwise fall back to label property.
+			String name = get_accessibility_name();
+			if (name.is_empty()) {
+				name = label;
+			}
+			DisplayServer::get_singleton()->accessibility_update_set_name(ae, name);
+			DisplayServer::get_singleton()->accessibility_update_set_description(ae, get_accessibility_description());
+			DisplayServer::get_singleton()->accessibility_update_set_num_value(ae, get_value());
+			DisplayServer::get_singleton()->accessibility_update_set_num_range(ae, get_min(), get_max());
+			if (get_step() > 0) {
+				DisplayServer::get_singleton()->accessibility_update_set_num_step(ae, get_step());
+			} else {
+				DisplayServer::get_singleton()->accessibility_update_set_num_step(ae, 1);
+			}
+
+			// Propagate accessibility name to internal LineEdit.
+			if (value_input && !name.is_empty()) {
+				value_input->set_accessibility_name(name);
+			}
+		} break;
+
 		case NOTIFICATION_ENTER_TREE: {
 			grabbing_spinner_speed = editing_integer ? EDITOR_GET("interface/inspector/integer_drag_speed") : EDITOR_GET("interface/inspector/float_drag_speed");
 			_update_value_input_stylebox();

editor/inspector/editor_inspector.cpp

@@ -289,8 +289,10 @@ void EditorProperty::_notification(int p_what) {
 
 			DisplayServer::get_singleton()->accessibility_update_set_role(ae, DisplayServer::AccessibilityRole::ROLE_BUTTON);
 
-			DisplayServer::get_singleton()->accessibility_update_set_name(ae, vformat(TTR("Property: %s"), label));
-			DisplayServer::get_singleton()->accessibility_update_set_value(ae, vformat(TTR("Property: %s"), label));
+			if (!label.is_empty()) {
+				DisplayServer::get_singleton()->accessibility_update_set_name(ae, vformat(TTR("Property: %s"), label));
+				DisplayServer::get_singleton()->accessibility_update_set_value(ae, vformat(TTR("Property: %s"), label));
+			}
 
 			DisplayServer::get_singleton()->accessibility_update_set_popup_type(ae, DisplayServer::AccessibilityPopupType::POPUP_MENU);
 			DisplayServer::get_singleton()->accessibility_update_add_action(ae, DisplayServer::AccessibilityAction::ACTION_SHOW_CONTEXT_MENU, callable_mp(this, &EditorProperty::_accessibility_action_menu));
@@ -715,6 +717,23 @@ void EditorProperty::_notification(int p_what) {
 void EditorProperty::set_label(const String &p_label) {
 	label = p_label;
 	queue_redraw();
+
+	// Propagate label to focusable child controls for accessibility.
+	// Also propagate to child EditorProperty nodes (e.g., wrapper -> inner property).
+	if (!label.is_empty()) {
+		for (Control *focusable : focusables) {
+			focusable->set_accessibility_name(label);
+		}
+		// Propagate to child EditorProperty focusables.
+		for (int i = 0; i < get_child_count(); i++) {
+			EditorProperty *child_ep = Object::cast_to<EditorProperty>(get_child(i));
+			if (child_ep && child_ep->get_label().is_empty()) {
+				for (Control *focusable : child_ep->focusables) {
+					focusable->set_accessibility_name(label);
+				}
+			}
+		}
+	}
 }
 
 String EditorProperty::get_label() const {
@@ -978,6 +997,12 @@ void EditorProperty::_focusable_focused(int p_index) {
 void EditorProperty::add_focusable(Control *p_control) {
 	p_control->connect(SceneStringName(focus_entered), callable_mp(this, &EditorProperty::_focusable_focused).bind(focusables.size()));
 	focusables.push_back(p_control);
+
+	// If property has focusable children, make the property itself not focusable
+	// to avoid duplicate tab stops.
+	if (get_focus_mode() != FOCUS_NONE) {
+		set_focus_mode(FOCUS_NONE);
+	}
 }
 
 void EditorProperty::grab_focus(int p_focusable) {
@@ -5950,6 +5975,7 @@ EditorInspector::EditorInspector() {
 	property_focusable = -1;
 
 	get_v_scroll_bar()->connect(SceneStringName(value_changed), callable_mp(this, &EditorInspector::_vscroll_changed));
+	get_v_scroll_bar()->set_accessibility_name(TTRC("Properties"));
 	update_scroll_request = -1;
 	if (EditorSettings::get_singleton()) {
 		refresh_countdown = float(EDITOR_GET("docks/property_editor/auto_refresh_interval"));

editor/inspector/editor_sectioned_inspector.cpp

@@ -367,6 +367,7 @@ SectionedInspector::SectionedInspector() :
 	sections->set_v_size_flags(SIZE_EXPAND_FILL);
 	sections->set_hide_root(true);
 	sections->set_theme_type_variation("TreeSecondary");
+	sections->set_accessibility_name(TTRC("Categories"));
 
 	left_vb->add_child(sections, true);
 

editor/settings/editor_settings_dialog.cpp

@@ -1141,6 +1141,9 @@ void EditorSettingsPropertyWrapper::setup(const String &p_property, EditorProper
 	editor_property = p_editor_property;
 	add_child(editor_property);
 
+	// Wrapper is not focusable - focus goes directly to the actual control.
+	set_focus_mode(FOCUS_NONE);
+
 	_update_override();
 
 	connect(SNAME("property_overridden"), callable_mp(this, &EditorSettingsPropertyWrapper::_create_override));
@@ -1169,7 +1172,9 @@ bool EditorSettingsInspectorPlugin::parse_property(Object *p_object, const Varia
 	real_property->set_name_split_ratio(0.0);
 	editor->setup(property, real_property, p_hint, p_hint_text, p_usage);
 
-	add_property_editor(p_path, editor);
+	// Generate label from property path for accessibility.
+	String label = EditorPropertyNameProcessor::get_singleton()->process_name(p_path, EditorPropertyNameProcessor::STYLE_CAPITALIZED, property, "");
+	add_property_editor(p_path, editor, false, label);
 	current_object = nullptr;
 	return true;
 }