-
-
Notifications
You must be signed in to change notification settings - Fork 957
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
style: Update launcher brand icons #2345
Conversation
Thanks, It looks like this is your first PR. I have a question, where did you get the social icons from? Unfortunately, some look to be of lesser quality than the old ones |
Thank you for reply, well, I get them from the official brand pages, then resize it |
Could we use SVG, if XAML supports that? |
https://uxwing.com/x-social-media-round-icon/ They also provide SVG if you want to try that route and see how they are. |
@minktusk Oh, I see. Like @Doprez suggested, try to find SVG images and try to export them into png files (i.e using Inkscape) - this way you can be sure that the images will be rendered correctly :) |
If WPF supports SVG file format then you can also replace pngs directly if you wish so. It shouldn't cause much lag |
Thanks, I will try change the icons to SVG and see if it works properly! |
Sorry for taking so long to update this pull request. I wasn't able to make it run SVG files. Honestly, I don't know if it's because XAML doesn't have support for it, or because I just don't know how to use XAML with SVG. So instead of using SVG, I just export them all to PNG using Inkscape ( Recommended by @Jklawreszuk ). PS: I also changed all mentions in the code from Twitter to XTwitter! |
I found this (also as NuGet) but have no experience with it, also not sure if there are other alternatives. Probably, we should keep whatever has less gears and is easier to maintain? |
Not sure about svg but xaml supports embedded vectors, see https://github.com/stride3d/stride/blob/master/sources/editor/Stride.Assets.Presentation/View/ImageDictionary.xaml |
Thanks ! |
PR Details
Replace old brand icons for new brand icons
Related Issue
Any, just some icons change
Types of changes
Checklist