-
Notifications
You must be signed in to change notification settings - Fork 66
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
System icon support #109
Comments
Why was legacy added? |
legacy means the bash script.... seems that a Go version is under preparation ;) |
*Rust, but yes |
Legacy refers to the shell script version, which as you saw is kind of hard to maintain |
Is the rewrite still happening? The |
No problem. I'm happy to help in any way that I can. I created FFI bindings to freerdp3 as that seems a better solution to using external processes everywhere: https://github.com/NaniNoni/freerdp-sys. I'm currently rewriting the winapps installer in Rust, as it's an independent component. |
I noticed that the install script favours system icons vs the present .svg files.
However, I didn't have the icons installed myself.
Eventually I edited the script to use the svg instead.
Maybe a check for the system icon can be added or let the user decide?
The text was updated successfully, but these errors were encountered: