Fix unable to build and update UI to Fluent style#15
Conversation
rmboggs
left a comment
rmboggs
left a comment
There was a problem hiding this comment.
Please see about the targetframework suggestion before approval.
| </PropertyGroup> | ||
| <PropertyGroup> | ||
| <OutputType>WinExe</OutputType> | ||
| <TargetFramework>net9.0-windows7.0</TargetFramework> |
There was a problem hiding this comment.
Would it be possible to have this target both net8 and net9? I ask because I think support for net9 will be EOL before net8.
Also, not to nitpick but can we get rid of the space changes?
There was a problem hiding this comment.
Sorry about the spacing, I have an extension that does that automatically. I've reverted the MainWindow to your previous styling and formatted the incoming changes to better match as well in b546eb9.
As for the NET8 downgrade, I tested it and am unable to do this without reverting everything back to the old Win7 stock theme. As a compromise, when NET10 releases in a few months, I can update the project again with another PR. Would that be acceptable?
There was a problem hiding this comment.
Reverted the project file spacing too (7f46785). Let me know if I missed anything.
|
@rmboggs I've upgraded the project to NET10 since it's finally hit release. It will have support out to Nov 2028. Let me know if this doesn't address your concerns. |
2 participants
Fixes #14
Additionally, this updates the UI to the modern fluent style added in NET9.
Dark:
Light: