Skip to content
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

New icons #66 #83

Merged
merged 4 commits into from
Apr 19, 2022
Merged

New icons #66 #83

merged 4 commits into from
Apr 19, 2022

Conversation

SychicBoy
Copy link

@SychicBoy SychicBoy commented Apr 18, 2022

Taskbar

  • Dark:
    image
  • Light:
    image

Explorer

  • Dark:
    image
    image
  • Light:
    image
    image

Start Menu

  • Dark:
    image
  • Light:
    image

@ElektroKill
Copy link
Member

Hi, are these files for sure valid .ico files? When I try to open them via the Windows Photo Viewer app, it displays an error that the file format is not supported. This does not happen with the icon files in place right now. How did you generate these icon files?

@SychicBoy
Copy link
Author

Hi, I just modified current icons.
If you could share your discord or telegram I'll send you .psd files.

@SychicBoy
Copy link
Author

SychicBoy commented Apr 18, 2022

Here is the .png files:

f740f03:

dnSpy
dnSpy-x86

7b9015f:

dnSpy-x86_brighter
dnSpy_brighter

@SychicBoy
Copy link
Author

I think I found the problem, Is that ok to I close current PR and create new one?

@ElektroKill
Copy link
Member

ElektroKill commented Apr 18, 2022

You can just update this pull request by adding commits to the respective branch in your repository. No need to create a new PR. My discord is ElektroKill#1949.

By the way, would it be possible to have the font for the x86 version be more similar to the current one? Also maybe we can make the outline a bit brighter since, in my opinion, it looks a bit dull right now.

@ElektroKill ElektroKill merged commit 97a11e7 into dnSpyEx:master Apr 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants