-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[v3/windows] Add an implementation of the SetIcon
method
#3153
Conversation
Important Auto Review SkippedAuto reviews are disabled on base/target branches other than the default branch. Please add the base/target branch pattern to the list of additional branches to be reviewed in the settings. Please check the settings in the CodeRabbit UI or the To trigger a single review, invoke the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChat with CodeRabbit Bot (
|
Woops! This change breaks the functionality of the |
You should be able to get the application using |
|
That'll teach me to answer on mobile 😂 Even easier! |
I'm assuming this is for runtime updates? Both icons indicated in the picture work fine in build |
Yes, this is updating icons at runtime.The screenshot shows the direct run |
If I'm not mistaken, wails/v3/pkg/application/application.go Line 493 in cf7537d
|
@almas1992 - Did you find a resolution for when you disable the icon? |
No, I haven't found an effective solution yet. 🫠 |
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Before:
![image](https://private-user-images.githubusercontent.com/9382335/292167660-90475e20-b914-4482-8f79-d9117ae51734.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzg5NDc2MjksIm5iZiI6MTczODk0NzMyOSwicGF0aCI6Ii85MzgyMzM1LzI5MjE2NzY2MC05MDQ3NWUyMC1iOTE0LTQ0ODItOGY3OS1kOTExN2FlNTE3MzQucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI1MDIwNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNTAyMDdUMTY1NTI5WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MDA2NGQxN2ViYTE2Yzg5NmMwMDRkMjMxZGYyMzQ0YmExNTE5NTdjMTNhOWZiMWI4ODMyODRhMjBjYmMzYTJmZiZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.TGbHUD1rU32XH1mjvZVuWuEYJbhVpgKNH2BuN1et1pw)
After:
![image](https://private-user-images.githubusercontent.com/9382335/292167876-347375ec-55ab-40d5-b098-96ccf3dc184f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzg5NDc2MjksIm5iZiI6MTczODk0NzMyOSwicGF0aCI6Ii85MzgyMzM1LzI5MjE2Nzg3Ni0zNDczNzVlYy01NWFiLTQwZDUtYjA5OC05NmNjZjNkYzE4NGYucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI1MDIwNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNTAyMDdUMTY1NTI5WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MDM2NmY1ZDkzMDNmYTZmZjA2MWRmMzMxYjUzOGUyMWZhNDZmMjNkN2QxNjc5NmFmYTU3YWQyZGU4OTFjNDA0MyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.D_oyvvnHqyImibgSXlRW0wdjR9CqkiXJsZotGsajPH4)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration using
wails doctor
.Test Configuration
Please paste the output of
wails doctor
. If you are unable to run this command, please describe your environment in as much detail as possible.Checklist:
website/src/pages/changelog.mdx
with details of this PR