Skip to content

Make sample app link more prominent in readme and update component list#789

Open
michael-hawker wants to merge 1 commit intomainfrom
michael-hawker-patch-1
Open

Make sample app link more prominent in readme and update component list#789
michael-hawker wants to merge 1 commit intomainfrom
michael-hawker-patch-1

Conversation

@michael-hawker
Copy link
Member

Updated readme with more prominent links to the sample app.

Add missing components that were added recently to the component list.

- [MarkdownTextBlock](https://github.com/CommunityToolkit/Labs-Windows/blob/main/components/MarkdownTextBlock/samples/MarkdownTextBlock.md)
- [Marquee](https://github.com/CommunityToolkit/Labs-Windows/blob/main/components/Marquee/samples/Marquee.md)
- [Notifications](https://github.com/CommunityToolkit/Labs-Windows/tree/main/components/Notifications)
- [OpacityMaskView](https://github.com/CommunityToolkit/Labs-Windows/tree/main/components/OpacityMaskView)
Copy link
Member

@Arlodotexe Arlodotexe Mar 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one fix-- this component has a .md file with the name of the component, which canonically serves as the index.md when published to mslearn.

Suggested change
- [OpacityMaskView](https://github.com/CommunityToolkit/Labs-Windows/tree/main/components/OpacityMaskView)
- [OpacityMaskView](https://github.com/CommunityToolkit/Labs-Windows/tree/main/components/OpacityMaskView/samples/OpacityMaskView.md)

The other component paths changed here don't have samples/docs or otherwise (in the case of ColorAnalyzer) have multiple sample docs but no component-level concept doc.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, there wasn't much in the doc as the sample isn't embedded when viewing from GitHub, so wasn't sure if it was as worthwhile, thoughts? (Would be nice if we got the direct linking for toolkitlabs.dev working)

Copy link
Member

@Arlodotexe Arlodotexe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor fix, otherwise looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants