NumberBox does not set a default AutomationProperties.Name value based on its header.

[YuliKl]

Describe the bug

Steps to reproduce the bug

Steps to reproduce the behavior:

1. Launch XAML Controls Gallery.
2. Activate Automation Properties from home page.
3. Navigate to text box present beside Font Size of texts and observe its name property.

Expected behavior

Name Property should be there for text box

Actual behavior

Text box name property missing.

Screenshots

Version Info

Gallery version (found on Settings page):

1.2.12.0

Additional context

Copied from internal bug

[marcelwgn]

I think this is an issue with NumberBox. Setting AutomationProperties.Name explicitly, the edit still has no name set. It seems like the UIA name is only applied to the "container" of the edit.

[YuliKl]

@chingucoding you're right. I remember having this discussion on the original NumberBox page PR. Moving this issue to WinUI.

[StephenLPeters]

@SavoySchuler If there is no header, what should the default automation name be? should the placeholder text be used? Header is an Object, what if it doesn't have a good string equivalent?

[ranjeshj]

We could do something similar to

microsoft-ui-xaml/dev/TreeView/TreeViewNode.cpp

Line 271 in 880dd22

hstring TreeViewNode::GetContentAsString()

[marcelwgn]

@SavoySchuler If there is no header, what should the default automation name be? should the placeholder text be used? Header is an Object, what if it doesn't have a good string equivalent?

When you look at the sample in the XAML Controls Gallery, the "spinner" has a UIA name, when using AutomationProperties.LabeledBy or AutomationProperties.Name. I think this UIA name should also apply to the "edit".

[robloo]

I don't think this issue was actually fixed. Can someone re-check it? I still see it in version 2.4.2.

Edit: Actually this is related to the header only so perhaps it was fixed. I'm having a related issue in #2951 though.