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.

Sign up for GitHub

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

Jump to bottom

SystemThemeWatcher Documentation content changes #1200

Merged
merged 2 commits into from
Aug 15, 2024
Merged

SystemThemeWatcher Documentation content changes #1200

merged 2 commits into from
Aug 15, 2024

Conversation

bayeggex
Copy link
Contributor

Pull request type

Please check the type of change your PR introduces:

  • Update
  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes

What is the current behavior?

Issue Number: N/A

What is the new behavior?

  • Added comprehensive documentation for the SystemThemeWatcher class.
  • Included details on features, usage, methods, and setup.
  • Provided example code for implementing SystemThemeWatcher in a WPF application.
  • Clarified instructions for setting up WPF UI resources in App.xaml.
  • Added a note about handling potential exceptions when using UnWatch.

Other information

N/A

@bayeggex bayeggex requested a review from pomianowski as a code owner August 14, 2024 13:28
@github-actions github-actions bot added documentation Improvements or additions to documentation PR Pull request release labels Aug 14, 2024
@pomianowski pomianowski merged commit 4ac9686 into lepoco:main Aug 15, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pomianowski pomianowski pomianowski approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation PR Pull request release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bayeggex @pomianowski