Hook up TokenizingTextBox AutoSuggestBoxStyle

[miksalmon]

Fixes #656

The AutoSuggestBoxStyle dependency property is not hooked up to the AutoSuggestBox's style in TokenizingTextBoxItemTextStyle. Is uses the static resource SystemAutoSuggestBoxStyle.

This fix sets the AutoSuggestBox's style in TokenizingTextBoxItemTextStyle to the AutoSuggestBoxStyle of the TokenizingTextBox and sets the AutoSuggestBoxStyle's default value to that of SystemAutoSuggestBoxStyle.

PR Type

What kind of change does this PR introduce?

Bugfix

What is the current behavior?

TokenizingTextBox's AutoSuggestBoxStyle is not applied

What is the new behavior?

TokenizingTextBox's AutoSuggestBoxStyle is applied

PR Checklist

• [X ] Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
• [X ] Based off latest main branch of toolkit
• [ X] Tested code with current supported SDKs
• [ X] Contains NO breaking changes

[miksalmon]

@dotnet-policy-service agree company="Microsoft"

[michael-hawker]

Main comment is that the SystemAutoSuggestBoxStyle is defined here in this file compared to where it's now getting referenced in TokenizingTextBox.xaml. So, it'd probably have to move to be properly looked up?

However, I'm not even sure if this is needed anymore and if it's a remnant of system WUX xaml when we weren't building on top of WinUI 2, like we are now for UWP and WinUI 3 for WASDK. Like we may just be able to reference DefaultAutoSuggestBoxStyle now?

@miksalmon did you notice a difference before/after here?

FYI @niels9001 who may be able to comment here. Doing a diff between what we have and the system style it doesn't appear there's a ton of big differences. Mostly the alignment top vs. stretch.

[miksalmon]

@michael-hawker During testing, I confirmed it works by keeping it there. I tested by modifying SystemAutoSuggestBoxStyle and making sure this was getting applied properly, which it did.

It can probably look it up just fine because TokenizingTextBoxItem.AutoSuggestBox.xaml is imported in TokenizingTextBox.xaml where it's used.

I'll be honest, I didn't verify what the actual differences were. It's possible we could use the default style or the default SystemAutoSuggestBoxStyle instead and get rid of this redefined one.