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

Changed WPF windows to use TextBlocks instead of Labels, added CHANGELOG file #476

Merged
merged 3 commits into from
Jul 4, 2023

Conversation

PeterJarrettUK
Copy link
Contributor

As part of our application we use theming which changes windows colours and font styles, and we want to push our colour scheme down into the NetSparkle UI's easily.

By changing the text displays to TextBlocks we can just use TextBlock Attached Properties - for example to set a foreground colour or Italic etc.

I have included an example of this in the WPF example project.

I have also added a CHANGELOG.md file to resolve issue #475

…o allow TextBlock Attached Properties.

Added example to NetSparkle.Samples.NetFramework.WPF to show example of changing the font to Italic
…o allow TextBlock Attached Properties.

Added example to NetSparkle.Samples.NetFramework.WPF to show example of changing the font to Italic
@Deadpikle
Copy link
Collaborator

Thanks for your PR. Appreciate it. I don't have a problem with the change from Label to TextBlock. As I've put in a few other threads, I'm extremely busy right now, but I have a holiday all week next week and am planning to attend to a few things on this repo at that point. :-) Thanks again.

@Deadpikle Deadpikle merged commit 92bdd2c into NetSparkleUpdater:develop Jul 4, 2023
4 checks passed
@Deadpikle
Copy link
Collaborator

Thank you for this! Merged in. The Avalonia build was already using TextBlock, so this is good that both are more in-sync now in this small way.

Appreciate it!

@PeterJarrettUK
Copy link
Contributor Author

No problem, always happy to help if I can

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