-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add code gen for setting UnoIcon on app Window #133
Conversation
556f289
to
2e0fa48
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on my end and is working... I've just 2 comments, but they don't invalidate the PR
f12bd05
to
6e2835b
Compare
Co-authored-by: Pedro Jesus <pedrojesus.cefet@gmail.com> Update src/.nuspec/Uno.Resizetizer.targets Co-authored-by: Pedro Jesus <pedrojesus.cefet@gmail.com>
3863181
to
4b182b8
Compare
@dansiegel the PR is falling with the following message
Did we miss any reference? Or the namespace is different for WPF? |
Congrats on your first pull-request! We ❤ the people who are involved in this project, and we’d love to pay it forward by sending you some swag. If you have any feedback (or ideas how to improve Uno as a open-source project) please open a feedback issue. |
GitHub Issue (If applicable): #
PR Type
What kind of change does this PR introduce?
What is the current behavior?
The developer needs to add native code to set the app's icon on the window
What is the new behavior?
Resizetizer generates a cross platform parameterless extension method on the window allowing the developer to easily set the icon on the window based on the UnoIcon provided.
Also normalizes the white space