Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[UWP Renderer] Do not wrap textblock in ComboBoxItem for compact choiceset #8221

Merged
merged 4 commits into from
Jan 10, 2023

Conversation

anna-dingler
Copy link
Contributor

@anna-dingler anna-dingler commented Jan 10, 2023

Related Issue

Fixes #7828

Description

Per the documentation, "Items added to the ComboBox are wrapped in ComboBoxItem containers.".

Since we were already wrapping the provided TextBlocks in ComboBoxItems before adding them to the ComboBox control, the items were wrapped twice, and Narrator did not handle this behavior correctly.

Adding TextBlocks to the ComboBox directly fixes this issue, and Narrator will announce "Snooze for 5 minutes ComboBox collapsed" as expected.

Sample Card

https://github.com/microsoft/AdaptiveCards/blob/main/samples/v1.5/Scenarios/CalendarReminder.json

How Verified

Verified manually on the UWP Visualizer.

Microsoft Reviewers: Open in CodeFlow

@beervoley
Copy link
Contributor

/azp run ios-ci.yml

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@anna-dingler anna-dingler merged commit a304b52 into main Jan 10, 2023
@anna-dingler anna-dingler deleted the anna/selectedInput branch January 10, 2023 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants