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

[Samples] Add headings to sample cards #8045

Merged
merged 11 commits into from Nov 21, 2022
Merged

[Samples] Add headings to sample cards #8045

merged 11 commits into from Nov 21, 2022

Conversation

anna-dingler
Copy link
Contributor

@anna-dingler anna-dingler commented Nov 10, 2022

Related Issue

Fixes #7607

Description

Add headings to sample cards.

I updated and moved required cards to v1.5 so that we could include TextBlock.style. Added TextBlock.heading to all cards already at v1.5 and above.

How Verified

Verified on the AdaptiveCards site.

image

Outstanding issue

Updating to v1.5 exposed a Table rendering issue in UWP - #8051

Microsoft Reviewers: Open in CodeFlow

@ghost ghost added the no-recent-activity label Nov 17, 2022
@ghost
Copy link

ghost commented Nov 17, 2022

Hi @anna-dingler. This pull request has had no recent activity for the past 5 days . Please take the necessary actions (review, address feedback or commit if reviewed already) to move this along.

* WIP: update project files for samples

* Update 1.5 references
@ghost ghost removed the no-recent-activity label Nov 17, 2022
@ghost
Copy link

ghost commented Nov 17, 2022

Staleness reset by anna-dingler

@anna-dingler anna-dingler merged commit 964ac35 into main Nov 21, 2022
@anna-dingler anna-dingler deleted the anna/addHeadings branch November 21, 2022 15:28
rankush pushed a commit to rankush/AdaptiveCards that referenced this pull request May 8, 2024
* Add headings to samples mentioned by accessibility team

* Add headings to v1.5 samples

* Remove duplicates

* Move remaining samples to v1.5

* [Samples]Update test apps samples (microsoft#8052)

* WIP: update project files for samples

* Update 1.5 references

* Disable scneario testing for .net

* Fix android visualizer (microsoft#8062)

* Resolve PR comments
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