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

Update to .Net8 #32

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

DerekGooding
Copy link

Updated and refactored everything that was out of date or depreciated.

I've tested and it works perfectly with .Net8. Just copy the Themes folder into the project and use the instructions from the original readme.

Just a quick update with the VS upgrade assistant
Error: Cannot create a default one-way converter.

So I created one.
Depreciated code replaced
As per shivan's pull request on the main repo.

Names were incorrect in Themes\ControlColors
Removed a redundant BasedOn
Code Maids + general refactoring cleanup
Since the desired import method is to copy the entire Themes folder into a new project, converters needs to be in there.
Reversing a cleanup change that broke something in MainWindow
@HiMarioLopez
Copy link

Thank you for this. I wish it were published to the main repo, but for now I will refer to you the new maintainer and be using your changes on your fork. :)

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